From 0fb47be9e7bc4da7beac799d5c691d76ce89944f Mon Sep 17 00:00:00 2001 From: Helene Arnesen Date: Mon, 6 Jan 2025 09:37:53 +0100 Subject: [PATCH 1/3] fix: catch validationException --- .../papirsykmelding/NasjonalSykmeldingService.kt | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/src/main/kotlin/no/nav/sykdig/digitalisering/papirsykmelding/NasjonalSykmeldingService.kt b/src/main/kotlin/no/nav/sykdig/digitalisering/papirsykmelding/NasjonalSykmeldingService.kt index 8d482d84..e7e51706 100644 --- a/src/main/kotlin/no/nav/sykdig/digitalisering/papirsykmelding/NasjonalSykmeldingService.kt +++ b/src/main/kotlin/no/nav/sykdig/digitalisering/papirsykmelding/NasjonalSykmeldingService.kt @@ -7,6 +7,7 @@ import no.nav.sykdig.LoggingMeta import no.nav.sykdig.applog import no.nav.sykdig.config.kafka.OK_SYKMELDING_TOPIC import no.nav.sykdig.digitalisering.exceptions.SykmelderNotFoundException +import no.nav.sykdig.digitalisering.exceptions.ValidationException import no.nav.sykdig.digitalisering.felles.Sykmelding import no.nav.sykdig.digitalisering.helsenett.SykmelderService import no.nav.sykdig.digitalisering.papirsykmelding.api.RegelClient @@ -82,7 +83,11 @@ class NasjonalSykmeldingService( ), StructuredArguments.fields(loggingMeta), ) - checkValidState(smRegistreringManuell, sykmelder, validationResult) + try { + checkValidState(smRegistreringManuell, sykmelder, validationResult) + } catch (e: ValidationException) { + log.error("catched validationException for oppgaveId $oppgaveId", e) + } val dokumentInfoId = oppgave.dokumentInfoId val journalpostId = oppgave.journalpostId From b1c29713c7916c6d52780dd753aa878fa6df2fba Mon Sep 17 00:00:00 2001 From: Helene Arnesen Date: Mon, 6 Jan 2025 09:51:36 +0100 Subject: [PATCH 2/3] fix: add ValidationException to ExceptionHandler --- .../digitalisering/exceptions/GlobalExceptionHandler.kt | 6 ++++++ .../papirsykmelding/NasjonalSykmeldingService.kt | 6 +----- 2 files changed, 7 insertions(+), 5 deletions(-) diff --git a/src/main/kotlin/no/nav/sykdig/digitalisering/exceptions/GlobalExceptionHandler.kt b/src/main/kotlin/no/nav/sykdig/digitalisering/exceptions/GlobalExceptionHandler.kt index a69d0b88..d967c277 100644 --- a/src/main/kotlin/no/nav/sykdig/digitalisering/exceptions/GlobalExceptionHandler.kt +++ b/src/main/kotlin/no/nav/sykdig/digitalisering/exceptions/GlobalExceptionHandler.kt @@ -67,6 +67,12 @@ class GlobalExceptionHandler { log.warn("Caught UnauthorizedException ${e.message}", e) return ResponseEntity.status(HttpStatus.UNAUTHORIZED).body("UnauthorizedException") } + + @ExceptionHandler(ValidationException::class) + fun handleUnAuthorizedException(e: ValidationException): ResponseEntity { + log.warn("Caught ValidationException ${e.message}", e) + return ResponseEntity.status(HttpStatus.BAD_REQUEST).body("ValidationException") + } } diff --git a/src/main/kotlin/no/nav/sykdig/digitalisering/papirsykmelding/NasjonalSykmeldingService.kt b/src/main/kotlin/no/nav/sykdig/digitalisering/papirsykmelding/NasjonalSykmeldingService.kt index e7e51706..11a5bf88 100644 --- a/src/main/kotlin/no/nav/sykdig/digitalisering/papirsykmelding/NasjonalSykmeldingService.kt +++ b/src/main/kotlin/no/nav/sykdig/digitalisering/papirsykmelding/NasjonalSykmeldingService.kt @@ -83,11 +83,7 @@ class NasjonalSykmeldingService( ), StructuredArguments.fields(loggingMeta), ) - try { - checkValidState(smRegistreringManuell, sykmelder, validationResult) - } catch (e: ValidationException) { - log.error("catched validationException for oppgaveId $oppgaveId", e) - } + checkValidState(smRegistreringManuell, sykmelder, validationResult) val dokumentInfoId = oppgave.dokumentInfoId val journalpostId = oppgave.journalpostId From e6ed4418ca51a5d76cea29c56d47920ec08ee087 Mon Sep 17 00:00:00 2001 From: Helene Arnesen Date: Mon, 6 Jan 2025 12:41:50 +0100 Subject: [PATCH 3/3] chore: warn to error --- .../sykdig/digitalisering/exceptions/GlobalExceptionHandler.kt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/main/kotlin/no/nav/sykdig/digitalisering/exceptions/GlobalExceptionHandler.kt b/src/main/kotlin/no/nav/sykdig/digitalisering/exceptions/GlobalExceptionHandler.kt index d967c277..bdd60307 100644 --- a/src/main/kotlin/no/nav/sykdig/digitalisering/exceptions/GlobalExceptionHandler.kt +++ b/src/main/kotlin/no/nav/sykdig/digitalisering/exceptions/GlobalExceptionHandler.kt @@ -70,7 +70,7 @@ class GlobalExceptionHandler { @ExceptionHandler(ValidationException::class) fun handleUnAuthorizedException(e: ValidationException): ResponseEntity { - log.warn("Caught ValidationException ${e.message}", e) + log.warn("Caught ValidationException ${e.message} validationresult ${e.validationResult}", e) return ResponseEntity.status(HttpStatus.BAD_REQUEST).body("ValidationException") } }