B - Retter bug der vi kaster en exception ved henting av info fra KRR

Co-authored-by: Mariam Pervez <[email protected]>
navikt · Nov 19, 2024 · 6a8e76b · 6a8e76b
1 parent 266c756
commit 6a8e76b
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 14 deletions.
diff --git a/src/main/kotlin/no/nav/eessi/pensjon/prefill/KrrService.kt b/src/main/kotlin/no/nav/eessi/pensjon/prefill/KrrService.kt
@@ -25,8 +25,8 @@ class KrrService(private val krrRestTemplate: RestTemplate,
     }
 
     //Henter inn telefonnummer og epostadresse fra KRR for å preutfylle SED
-    fun hentPersonFraKrr(personIdent: String, inkluderSikkerDigitalPost: Boolean?= false): KrrPerson {
-        return HentPerson.measure {
+    fun hentPersonFraKrr(personIdent: String, inkluderSikkerDigitalPost: Boolean?= false): KrrPerson? {
+        HentPerson.measure {
             val url = "/rest/v1/person?inkluderSikkerDigitalPost=$inkluderSikkerDigitalPost"
             logger.debug("Henter informasjon fra KRR: $url")
 
@@ -45,15 +45,11 @@ class KrrService(private val krrRestTemplate: RestTemplate,
 
                 logger.debug("Hent person fra KRR: response: ${response.body}".trimMargin())
 
-                response.body?.let {
-                    mapJsonToAny<KrrPerson>(it)
-                } ?: throw IllegalArgumentException("Mangler melding fra KRR")
+                return@measure response.body?.let { mapJsonToAny<KrrPerson>(it) } ?: throw IllegalArgumentException("Mangler melding fra KRR")
             } catch (e: HttpClientErrorException.NotFound) {
-                throw IllegalArgumentException("Person: $personIdent ikke funnet (404)")
-            } catch (e: Exception) {
-                throw e
+                logger.error("Person: $personIdent ikke funnet (404)")
             }
         }
-
+        return null
     }
 }
diff --git a/src/main/kotlin/no/nav/eessi/pensjon/prefill/PrefillService.kt b/src/main/kotlin/no/nav/eessi/pensjon/prefill/PrefillService.kt
@@ -87,9 +87,9 @@ class PrefillService(
     private fun hentKrrPerson(norskIdent: String, request: ApiRequest): PersonInfo {
         val krrPerson = krrService.hentPersonFraKrr(norskIdent).let { personResponse ->
             KrrPerson(
-                reservert = personResponse.reservert,
-                epostadresse = personResponse.epostadresse.validateEmail(request.processDefinitionVersion),
-                mobiltelefonnummer = personResponse.mobiltelefonnummer
+                reservert = personResponse?.reservert,
+                epostadresse = personResponse?.epostadresse.validateEmail(request.processDefinitionVersion),
+                mobiltelefonnummer = personResponse?.mobiltelefonnummer
             )
         }
 
@@ -105,8 +105,7 @@ class PrefillService(
                 krrPerson.reservert,
                 krrPerson.epostadresse,
                 krrPerson.mobiltelefonnummer
-            )
-                .also { logger.info("Hentet telefon og epost fra KRR: ${krrPerson.toJson()}") }
+            ).also { logger.info("Hentet telefon og epost fra KRR: ${krrPerson.toJson()}") }
         }
         return personInfo
     }