navikt/nav-dekoratoren-api

nav-dekoratoren-api

Ktor-backend for dekoratøren på nav.no.

Secrets ligger i Google Secret Manager.

Deploy til dev

Actions -> Velg workflow -> Run workflow -> Velg branch -> Run workflow

Prodsetting

• Lag en PR til master, og merge inn etter godkjenning
• Lag en release på master med versjon-bump, beskrivende tittel og oppsummering av endringene dine
• Publiser release'en for å starte deploy til prod

Logging

Kibana

Henvendelser

Spørsmål knyttet til koden eller prosjektet kan rettes mot https://github.com/orgs/navikt/teams/personbruker

For NAV-ansatte

Interne henvendelser kan sendes via Slack i kanalen #team-personbruker.

nav-dekoratoren-api's Issues

Undersøke feil som oppstår sporadisk

Denne feilen spys ut i loggene med ujevne mellomrom:

Failed to submit a listener notification task. Event loop shut down?

Mistenker at det skyldes bruk av runBlocking. Har spurt om tips her: https://nav-it.slack.com/archives/C9T4KG12M/p1662986928953379

Bruke Caffeine for caching

I dag bruker vi litt hjemmesnekra greier for caching. Dette bør byttes ut med Caffeine: https://github.com/ben-manes/caffeine

DittNAV bruker dette i sine prosjekter.

Vi bør skrive oss bort fra dittnav-dependencies i samme slengen.

Innloggingsstatus: Fjerne ubrukt OpenAM-kode

Eksponere summary-objektet i auth-endepunkt

Eksponere følgende data i auth-endepunkt:

• sessionExpires
• tokenExpires
• isSessionActive

Rename til nav-dekoratoren-api

Team min side ønsker å bruke innloggingsstatus for å hente varsler til dekoratøren. Tanken er at da at innloggingsstatus blir en mer generell backend-app for dekoratøren. Den bør da få et mer passende navn.

Vi kan vente med renaming til de har gjort ferdig sine endringer, slik at vi slipper conflicts.

Husk å oppdatere urler i dekoratøren.

Refaktorer - package by feature.

Flytte unleash-endepunkt fra pb-unleash-proxy

I dag har vi en app pb-unleash-proxy som kun inneholder ett (lite brukt) endepunkt for feature toggles. I tråd med at vi planlegger å gjøre innloggingsstatus om til en mer generell backend-app for dekoratøren, gir det mening å flytte denne funksjonaliteten over til denne. Da kan vi arkivere pb-unleash-proxy.

Bumpe dependenices og endre oppsett

I dag bruker vi DittNavs dependency-repo for å velge versjoner av dependencies i innloggingsstatus. Dette er uheldig da oppsettet krever at vi må koordinere med dem for å bumpe dependencies eller legge til nye. I stedet bør vi ha en Versions.kt-fil i buildSrc-mappen til innloggingsstatus som inneholder de versjonene vi ønsker å bruke.

Flere av de dependenciene vi bruker i dag er utdaterte og bør bumpes i samme slengen.

Kotlin-dsl med buildSrc-oppsettet gir ikke automatisk highlighting av utdaterte versjoner. I stedet kan vi bruke følgende plugin: https://github.com/ben-manes/gradle-versions-plugin

Utbedre kryptisk feillogging

Feil mot PDL fremstår som serialiseringsfeil, men er sannsynligvis dårlig håndtering av uventet respons. Dette er sannsynligvis en timeout feks: https://logs.adeo.no/app/discover#/doc/96e648c0-980a-11e9-830a-e17bbd64b4db/logstash-apps-prod-005940?id=UPX5oIUBGDDk0ymyTGdi&breadcrumb=%23%2F%3F_a%3D(columns%3A!(level%2Cmessage%2Cenvclass%2Capplication%2Cpod)%2Cfilters%3A!()%2Cindex%3A'96e648c0-980a-11e9-830a-e17bbd64b4db'%2Cinterval%3Aauto%2Cquery%3A(language%3Akuery%2Cquery%3A'application%3Anav-dekoratoren-api%2520AND%2520envclass%3Ap')%2Csort%3A!(!('%40timestamp'%2Cdesc)))%26_g%3D(filters%3A!()%2CrefreshInterval%3A(pause%3A!t%2Cvalue%3A0)%2Ctime%3A(from%3Anow-24h%252Fh%2Cto%3Anow))

Oppdatere deploy workflows tilsvarende de andre appene våre

Migrere innloggingsstatus til GCP

Oppdatere pdl-consumer

Denne bør kalle pdl direkte fra GCP og bruke tokenx. I dag bruker den loginservice-token, og går via api-gw.

Undersøke økt bruk av minne og CPU

Nav-dekoratoren-api har fått ganske kraftig økning i minne- og til dels CPU-bruk etter prodsetting 2. november, og vi fikk en OOM-alert 6. nov: https://nav-it.slack.com/archives/C02MC2QSF6D/p1667736122024019

Kan det ha noe å gjøre med disse endringene i varselbjelle-apiet? v2.0.4...v2.0.5

Migrere til Ktor 2.x.x

Krever siste versjon av token-support:

https://nav-it.slack.com/archives/C01381BAT62/p1655292931153169?thread_ts=1651663508.904229&cid=C01381BAT62

Gjelder følgende apper:

• Innloggingsstatus
• Dsop-api
• Personopplysninger-proxy
• Navno-api

Fikse logback-versjon og div opprydning

Innloggingsstatus har noen feilmeldinger i loggene som skyldes stack overflow. Dette virker å stamme fra en bug i logback-versjonen som brukes, som er fikset i nyere versjoner. Denne bør derfor bumpes.

Sannsyligvis er det flere gamle dependencies som bør bumpes, og ser også litt ubrukte imports og div snacks her og der. Kjør "inspect code" i IntelliJ og rydd opp der det trengs.

Flytte innloggingsstatus-ingresser over til ny app

Nais-teamet jobber med å gå over på nye ingresser, men dersom det fortsatt er trafikk på innloggingsstatus-ingressene etter dette, så må de flyttes over til nav-dekoratoren-api slik at de fortsatt støttes når vi sletter den gamle deploymenten.

Husk også å slette gamle secrets osv.