Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Hent saksbehandler navn fra integrasjoner #5088

Merged
merged 4 commits into from
Feb 20, 2025

Conversation

UyQuangNguyen
Copy link
Contributor

@UyQuangNguyen UyQuangNguyen commented Feb 18, 2025

Favrokort: https://favro.com/organization/98c34fb974ce445eac854de0/1844bbac3b6605eacc8f5543?card=NAV-24034

Det er ønskelig at navn i vedtaksbrev på saksbehandler og beslutter kommer på formatet "Fornavn Mellomnavn Etternavn".
Per nå henter vi ut navn fra tokenet, og vi får dette: "Etternavn, Fornavn Mellomnavn".

Jeg har derfor undersøkt 3 forskjellige måter å få dette ordnet på:

  • Vi splitter navn fra token på "," og restrukturerer det vi får fra tokenet
    Jeg hadde egentlig ikke lyst til å gå for denne retningen, da det føles feil ut å tukle med navnet vi får fra tokenet.
    Dessuten så er det ikke gitt at navn vil alltid komme med denne strukturen i evig tid, og dersom det en dag endres og kommaen forsvinner, så vil ting feile.
  • Legge på ekstra claims i tokenet og hente fra family_name og given_name
    Det er mulig å hente dette i AD tokenet, men family name og given name er optional claims som vanligvis ikke følges med i tokenet. Se https://learn.microsoft.com/en-us/entra/identity-platform/access-token-claims-reference. Jeg sjekket med Trong fra Nais teamet, og dersom vi ønsker dette i tokenet så må det endres i flere systemer, og han kunne ikke si noe konkret på når dette kunne bli gjort. Det ble derfor anbefalt å bruke Graph APIet til Microsoft for å hente det ut.
  • Gjøre kall mot familie-integrasjoner
    Dette er løsningen jeg gikk for.
    I familie integrasjoner har vi et endepunkt for å hente ut saksbehandler objekt. Det bruker Graph APIeet. Dette objektet inkluderer navna, som jeg nå henter og slår sammen. Hvis dette feiler fallbacker vi til vanlig prosedyre og henter ut navn fra token.

@UyQuangNguyen UyQuangNguyen requested a review from a team as a code owner February 18, 2025 23:31
Copy link
Contributor

@bragejahren bragejahren left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Enig i analysen din! Supert med fallback til gammel prosedyre dersom kallet mot integrasjoner feiler 👍 ✅

@UyQuangNguyen UyQuangNguyen added this pull request to the merge queue Feb 20, 2025
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
64.0% Coverage on New Code (required ≥ 70%)

See analysis details on SonarQube Cloud

Merged via the queue into main with commit 215a038 Feb 20, 2025
7 of 8 checks passed
@UyQuangNguyen UyQuangNguyen deleted the Hent_saksbehandler_navn_fra_integrasjoner branch February 20, 2025 23:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants