Saksbehandlerprofil: bytt fra ldap til fp-tilgang som kilde (#2101)

.deploy/dev-fss-teamforeldrepenger.json

@@ -4,7 +4,6 @@
   "oracleconfigkv":  "/oracle/data/dev/config/fplos_q1",
   "oraclecredskv": "/oracle/data/dev/creds/fplos_q1-fplos_q1",
   "serviceuserkv": "/serviceuser/data/dev/srvfplos",
-  "ldapuserkv": "/serviceuser/data/dev/srvssolinux",
   "ingresses":  [
     "https://fplos.dev-fss-pub.nais.io"
   ],

.deploy/naiserator.yaml

@@ -54,8 +54,6 @@ spec:
         mountPath: /var/run/secrets/nais.io/defaultDSconfig
       - kvPath: {{serviceuserkv}}
         mountPath: /var/run/secrets/nais.io/serviceuser
-      - kvPath: {{ldapuserkv}}
-        mountPath: /var/run/secrets/nais.io/ldap
   azure:
     application:
       enabled: true

.deploy/prod-fss-teamforeldrepenger.json

@@ -4,7 +4,6 @@
   "oracleconfigkv":  "/oracle/data/prod/config/fplos_p",
   "oraclecredskv": "/oracle/data/prod/creds/fplos_p-fplos_p",
   "serviceuserkv": "/serviceuser/data/prod/srvfplos",
-  "ldapuserkv": "/serviceuser/data/prod/srvssolinux",
   "ingresses":  [
     "https://fplos.prod-fss-pub.nais.io"
   ],

src/main/java/no/nav/foreldrepenger/los/organisasjon/ansatt/AnsattTjeneste.java

@@ -1,14 +1,10 @@
 package no.nav.foreldrepenger.los.organisasjon.ansatt;
 
-import java.time.Duration;
 import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
 import java.util.concurrent.TimeUnit;
 
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
-
 import jakarta.enterprise.context.ApplicationScoped;
 import jakarta.inject.Inject;
 import no.nav.foreldrepenger.los.domene.typer.aktør.OrganisasjonsEnhet;
@@ -19,8 +15,6 @@
 @ApplicationScoped
 public class AnsattTjeneste {
 
-    private static final Logger LOG = LoggerFactory.getLogger(AnsattTjeneste.class);
-
     private static final LRUCache<String, BrukerProfil> ANSATT_PROFIL = new LRUCache<>(1000, TimeUnit.MILLISECONDS.convert(24 * 7, TimeUnit.HOURS));
     private static final LRUCache<String, List<String>> ANSATT_ENHETER = new LRUCache<>(1000, TimeUnit.MILLISECONDS.convert(25, TimeUnit.HOURS));
     private static final Map<String, String> ENHETSNUMMER_AVDELINGSNAVN_MAP = new HashMap<>();
@@ -30,8 +24,6 @@ public class AnsattTjeneste {
     private List<String> aktuelleEnhetIder;
 
 
-
-
     AnsattTjeneste() {
         // for CDI proxy
     }
@@ -48,37 +40,12 @@ public BrukerProfil hentBrukerProfil(String ident) {
         if (ANSATT_PROFIL.get(ident) == null) {
             //TODO:  Her bør vi egentlig tenke om NOM er ikke riktigere å bruke - bør være raskere å slå opp navn og epost.
             // Jeg har sjekket med NOM (01.07.2024) og de støtter en så lenge ikke Z-identer i dev. Men prod brukere er tilgjengelig.
-            var før = System.nanoTime();
-            var ldapRespons = new LdapBrukeroppslag().hentBrukerProfil(ident);
-            LOG.info("LDAP bruker profil oppslag: {}ms. ", Duration.ofNanos(System.nanoTime() - før).toMillis());
-            var ansattEnhet = avdeling(ldapRespons.ansattEnhet());
-            if (ansattEnhet == null) {
-                LOG.info("PROFIL LDAP: brukers enhet {} ikke blant saksbehandlingsenhetene", ldapRespons.ansattEnhet());
-            }
-            var brukerProfil = new BrukerProfil(ldapRespons.ident(), ldapRespons.navn(), ldapRespons.fornavnEtternavn(), ldapRespons.epostAdresse(),
-                ansattEnhet);
-            sammenlignMedAzureGraphFailSoft(ident, brukerProfil);
+            var brukerProfil = mapTilDomene(new AzureBrukerKlient().brukerProfil(ident));
             ANSATT_PROFIL.put(ident, brukerProfil);
         }
         return ANSATT_PROFIL.get(ident);
     }
 
-    private static void sammenlignMedAzureGraphFailSoft(String ident, BrukerProfil ldapBrukerInfo) {
-        LOG.info("PROFIL Azure. Henter fra azure.");
-        try {
-            var før = System.nanoTime();
-            var azureBrukerProfil = mapTilDomene(new AzureBrukerKlient().brukerProfil(ident));
-            if (!ldapBrukerInfo.equals(azureBrukerProfil)) {
-                LOG.info("PROFIL Azure. Profiler fra ldap og azure er ikke like. Azure: {} != LDAP: {}", azureBrukerProfil, ldapBrukerInfo);
-            } else {
-                LOG.info("PROFIL Azure. Azure == LDAP :)");
-            }
-            LOG.info("Azure bruker profil oppslag: {}ms. ", Duration.ofNanos(System.nanoTime() - før).toMillis());
-        } catch (Exception ex) {
-            LOG.info("PROFIL Azure. Klienten feilet med exception: {}", ex.getMessage());
-        }
-    }
-
     public List<String> hentAvdelingerNavnForAnsatt(String ident) {
         if (aktuelleEnhetIder == null) {
             aktuelleEnhetIder = organisasjonRepository.hentAktiveAvdelinger().stream().map(Avdeling::getAvdelingEnhet).toList();
@@ -96,7 +63,7 @@ public List<String> hentAvdelingerNavnForAnsatt(String ident) {
     }
 
     private static BrukerProfil mapTilDomene(AzureBrukerKlient.BrukerProfilResponse klientResponse) {
-        return new BrukerProfil(klientResponse.ident(), klientResponse.navn(), klientResponse.fornavnEtternavn(), klientResponse.epost(),
+        return new BrukerProfil(klientResponse.ident(), klientResponse.fornavnEtternavn(), klientResponse.fornavnEtternavn(), klientResponse.epost(),
             avdeling(klientResponse.ansattVedEnhetId()));
     }
 

src/main/java/no/nav/foreldrepenger/los/tjenester/felles/dto/ReservasjonStatusDtoTjeneste.java

@@ -44,7 +44,7 @@ private String hentNavn(String ident) {
         if (ident == null) {
             return null;
         }
-        return tryOrEmpty(() -> ansattTjeneste.hentBrukerProfil(ident), "ldap")
+        return tryOrEmpty(() -> ansattTjeneste.hentBrukerProfil(ident), "brukerprofil")
             .map(BrukerProfil::navn)
             .orElse("Ukjent");
     }

src/main/java/no/nav/foreldrepenger/los/tjenester/felles/dto/SaksbehandlerDtoTjeneste.java

@@ -67,9 +67,9 @@ private Optional<SaksbehandlerMedAvdelingerDto> tilSaksbehandlerMedAvdelingerDto
     }
 
     public SaksbehandlerMedAvdelingerDto lagKjentOgUkjentSaksbehandlerMedAvdelingerDto(Saksbehandler saksbehandler) {
-        // saksbehandler kan eksistere i basen men være ukjent i ldap
+        // saksbehandler kan eksistere i basen men være ukjent i azuread
         var ident = saksbehandler.getSaksbehandlerIdent();
-        var saksbehandlerDto = tilSaksbehandlerDto(ident);
+        var saksbehandlerDto = tilSaksbehandlerDto(ident.toLowerCase());
         if (saksbehandlerDto.isPresent()) {
             var avdelinger = ansattTjeneste.hentAvdelingerNavnForAnsatt(ident);
             return new SaksbehandlerMedAvdelingerDto(saksbehandlerDto.get(), avdelinger);

src/main/resources/application-dev-fss.properties

@@ -1,10 +1,3 @@
-# LDAP
-ldap.url=ldaps://ldapgw.preprod.local
-ldap.basedn=dc=preprod,dc=local
-ldap.domain=PREPROD.LOCAL
-ldap.user.basedn=ou=NAV,ou=BusinessUnits,dc=preprod,dc=local
-ldap.serviceuser.basedn=ou=ServiceAccounts,dc=preprod,dc=local
-
 # Database
 # defaultDS.username=<leses inn fra vault ved oppstart gjennom 05-import-users.sh>
 # defaultDS.password=<leses inn fra vault ved oppstart gjennom 05-import-users.sh>

src/main/resources/application-prod-fss.properties

@@ -1,10 +1,3 @@
-# LDAP
-ldap.url=ldaps://ldapgw.adeo.no
-ldap.basedn=dc=adeo,dc=no
-ldap.domain=ADEO.NO
-ldap.user.basedn=ou=NAV,ou=BusinessUnits,dc=adeo,dc=no
-ldap.serviceuser.basedn=ou=ServiceAccounts,dc=adeo,dc=no
-
 # Database
 # defaultDS.username=<leses inn fra vault ved oppstart gjennom 05-import-users.sh>
 # defaultDS.password=<leses inn fra vault ved oppstart gjennom 05-import-users.sh>

src/test/resources/application-vtp.properties

@@ -34,11 +34,6 @@ azure.app.well.known.url=http://authserver:8060/rest/azuread/.well-known/openid-
 AZURE_APP_CLIENT_ID=vtp
 NAIS_CLUSTER_NAME=vtp
 
-# LDAP
-ldap.url=ldap://localhost:8389/
-ldap.auth=none
-ldap.user.basedn=ou\=NAV,ou\=BusinessUnits,dc\=test,dc\=local
-
 task.manager.polling.wait=5
 task.manager.polling.delay=5
 task.manager.polling.tasks.size=1