PdlClient i mottak

[throndi]

Hvorfor ?

Vi ønsker ikke å bruker integrasjoner for å gjøre kall mot PDL, og trenger derfor en klient.

PR, integrasjoner: navikt/familie-integrasjoner#986

https://favro.com/organization/98c34fb974ce445eac854de0/a64c6aad9b0d61ef6c0290bd?card=NAV-20981

[ViktorGSolberg]

Hvorfor bruker vi ikke bare pdlClienten via integrasjoner? Da slipper man å legge til pdl-client i denne appen her også

[throndi]

Hvorfor bruker vi ikke bare pdlClienten via integrasjoner? Da slipper man å legge til pdl-client i denne appen her også

Tanken er vel å gå direkte mot PDL, i stedet for å gå via integrasjoner hvis mottak er den eneste appen som bruker det aktør endepunktet, sånn at vi kan fjerne dette fra integrasjoner og.

[olekvernberg]

Ser bra ut! Er det testet at integrasjonen virker i preprod?

[ViktorGSolberg]

Member

Joda, men i sum så fjerner man kanskje 90 linjer fra integrasjoner og legger til 400 her. I tillegg er det jo fortsatt slik alle de andre appene våre går via integrasjoner mot pdl så da blir det litt merkelig at akkurat dette kallet skal hoppe over integrasjoner

[throndi]

Member

Joda, men i sum så fjerner man kanskje 90 linjer fra integrasjoner og legger til 400 her. I tillegg er det jo fortsatt slik alle de andre appene våre går via integrasjoner mot pdl så da blir det litt merkelig at akkurat dette kallet skal hoppe over integrasjoner

ja, enig i at det blir mange ekstra kodelinjer for å legge til en pdl-klient (selv om linjene er ganske isolerte og for seg selv), samtidig som at det er kanskje litt rart å ha endepunkt i integrasjoner som bare én app bruker, og som nesten aldri brukes. Tanken var å fjerne disse aktørendepunktene pga det. Formålet med integrasjoner appen er kanskje ikke å være en "pdl-proxy" på det viset heller (ga kanskje mere mening med TPS) ? BA gikk over til at alle appene går mot PDL direkte i alle fall.

[charliemidtlyng]

Må innrømme at dette føles som en voldsom utvidelse for et marginalt behov.
I tillegg tror jeg halve behovet kan omgås ettersom Oppgave nå tillater å opprette oppgaver basert på personIdenter:
https://github.com/navikt/oppgave/commit/91432e35537e3ad4ddff5f292f67d79eee1d25dd

Det vi isåfall trenger "pdl" til er å hente ut fnr basert på aktørId for å finne behandlende enhet fra integrasjoner... Vi gjør samme greia i iverksett med at vi henter aktørid for personidenter.