From 24e6eb91fabad53c300a75ad0b44d0c7d90aff16 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C3=98yvind=20N=2E=20Wed=C3=B8e?=
 <oyvind.norsted.wedoe@nav.no>
Date: Tue, 10 Dec 2024 15:22:23 +0100
Subject: [PATCH] Remove retryable. Skip second ytelse access check in system
 context.

---
 .../no/nav/klage/oppgave/service/BehandlingService.kt       | 4 +++-
 .../no/nav/klage/oppgave/service/ExternalMottakFacade.kt    | 3 +--
 .../nav/klage/oppgave/service/KabalInnstillingerService.kt  | 6 ------
 3 files changed, 4 insertions(+), 9 deletions(-)

diff --git a/src/main/kotlin/no/nav/klage/oppgave/service/BehandlingService.kt b/src/main/kotlin/no/nav/klage/oppgave/service/BehandlingService.kt
index a63daa614..bcebe5cc8 100644
--- a/src/main/kotlin/no/nav/klage/oppgave/service/BehandlingService.kt
+++ b/src/main/kotlin/no/nav/klage/oppgave/service/BehandlingService.kt
@@ -667,7 +667,9 @@ class BehandlingService(
         if (tildeltSaksbehandlerIdent != null) {
             //Denne sjekken gjøres kun når det er en tildeling:
 
-            checkYtelseAccess(tildeltSaksbehandlerIdent = tildeltSaksbehandlerIdent, behandling = behandling)
+            if (!systemUserContext) {
+                checkYtelseAccess(tildeltSaksbehandlerIdent = tildeltSaksbehandlerIdent, behandling = behandling)
+            }
 
             //if fagsystem is Infotrygd also do this.
             if (behandling.shouldUpdateInfotrygd()) {
diff --git a/src/main/kotlin/no/nav/klage/oppgave/service/ExternalMottakFacade.kt b/src/main/kotlin/no/nav/klage/oppgave/service/ExternalMottakFacade.kt
index 47d22399b..f7d81bd0f 100644
--- a/src/main/kotlin/no/nav/klage/oppgave/service/ExternalMottakFacade.kt
+++ b/src/main/kotlin/no/nav/klage/oppgave/service/ExternalMottakFacade.kt
@@ -84,6 +84,7 @@ class ExternalMottakFacade(
                 saksbehandlerIdent = saksbehandlerIdent,
             )
         } catch (e: Exception) {
+            logger.error("Klarte ikke å tildele behandling ${behandling.id} til saksbehandlerIdent $saksbehandlerIdent. Feil: $e")
             mottakService.createTaskForMerkantil(
                 behandlingId = behandling.id,
                 reason = "Klarte ikke å tildele behandling ${behandling.id} til saksbehandlerIdent $saksbehandlerIdent. Feilmelding: ${e.message}"
@@ -149,6 +150,4 @@ class ExternalMottakFacade(
             systemUserContext = true,
         )
     }
-
-
 }
\ No newline at end of file
diff --git a/src/main/kotlin/no/nav/klage/oppgave/service/KabalInnstillingerService.kt b/src/main/kotlin/no/nav/klage/oppgave/service/KabalInnstillingerService.kt
index 67da46707..0dbe9ca29 100644
--- a/src/main/kotlin/no/nav/klage/oppgave/service/KabalInnstillingerService.kt
+++ b/src/main/kotlin/no/nav/klage/oppgave/service/KabalInnstillingerService.kt
@@ -4,14 +4,12 @@ import no.nav.klage.kodeverk.Ytelse
 import no.nav.klage.oppgave.clients.kabalinnstillinger.KabalInnstillingerClient
 import no.nav.klage.oppgave.clients.kabalinnstillinger.model.*
 import no.nav.klage.oppgave.domain.klage.Behandling
-import org.springframework.retry.annotation.Retryable
 import org.springframework.stereotype.Service
 
 @Service
 class KabalInnstillingerService(
     private val kabalInnstillingerClient: KabalInnstillingerClient,
 ) {
-    @Retryable
     fun getPotentialSaksbehandlere(behandling: Behandling): Saksbehandlere {
         return kabalInnstillingerClient.searchSaksbehandlere(
             SaksbehandlerSearchInput(
@@ -21,7 +19,6 @@ class KabalInnstillingerService(
         )
     }
 
-    @Retryable
     fun getPotentialMedunderskrivere(behandling: Behandling): Medunderskrivere {
         if (behandling.tildeling == null) {
             return Medunderskrivere(medunderskrivere = emptyList())
@@ -36,7 +33,6 @@ class KabalInnstillingerService(
         )
     }
 
-    @Retryable
     fun getPotentialROL(behandling: Behandling): Saksbehandlere {
         return kabalInnstillingerClient.searchROL(
             ROLSearchInput(
@@ -46,14 +42,12 @@ class KabalInnstillingerService(
     }
 
     //TODO: Bør vi ha et cache her? Kan være et problem om leder gir nye tilganger, kanskje et kortere cache?
-    @Retryable
     fun getTildelteYtelserForSaksbehandler(navIdent: String): List<Ytelse> {
         return kabalInnstillingerClient.getSaksbehandlersTildelteYtelser(navIdent).ytelseIdList.map {
             Ytelse.of(it)
         }
     }
 
-    @Retryable
     fun getTildelteYtelserForEnhet(enhet: String): Set<Ytelse> {
         return kabalInnstillingerClient.getTildelteYtelserForEnhet(enhet).ytelseIdList.map {
             Ytelse.of(it)