From 26477cacd7da9c606f4b68d4a39d024ba8fe5580 Mon Sep 17 00:00:00 2001 From: Magnus Rinnan Date: Thu, 15 Aug 2024 14:15:12 +0200 Subject: [PATCH] Lagt til error-logging til vanlig logg ved feil ved innhenting av grunnlag --- pom.xml | 5 +- .../service/HentArbeidsforholdService.kt | 82 ++++++++++--------- .../service/HentBarnetilleggService.kt | 25 ++++-- .../service/HentBarnetilsynService.kt | 26 ++++-- .../HentKontantst\303\270tteService.kt" | 25 ++++-- .../service/HentRelatertePersonerService.kt | 73 +++++++++++++---- .../grunnlag/service/HentSivilstandService.kt | 14 +++- .../service/HentSkattegrunnlagService.kt | 12 +-- ...netrygdOgSm\303\245barnstilleggService.kt" | 20 +++-- .../service/InntektskomponentenService.kt | 45 +++++++--- 10 files changed, 227 insertions(+), 100 deletions(-) diff --git a/pom.xml b/pom.xml index f72bd8d3..37e5d440 100644 --- a/pom.xml +++ b/pom.xml @@ -21,10 +21,11 @@ 1.9.24 3.2.0 - 4.1.7 + 3.2.0 + 1.20230511093912_dc35d81 - 2024.07.01.144736 + 2024.08.15.122550 1.10.4 5.3.1 2.5.0 diff --git a/src/main/kotlin/no/nav/bidrag/grunnlag/service/HentArbeidsforholdService.kt b/src/main/kotlin/no/nav/bidrag/grunnlag/service/HentArbeidsforholdService.kt index 0a5a40ca..5cfe6199 100644 --- a/src/main/kotlin/no/nav/bidrag/grunnlag/service/HentArbeidsforholdService.kt +++ b/src/main/kotlin/no/nav/bidrag/grunnlag/service/HentArbeidsforholdService.kt @@ -3,6 +3,7 @@ package no.nav.bidrag.grunnlag.service import no.nav.bidrag.domene.enums.grunnlag.GrunnlagRequestType import no.nav.bidrag.grunnlag.SECURE_LOGGER import no.nav.bidrag.grunnlag.consumer.arbeidsforhold.ArbeidsforholdConsumer +import no.nav.bidrag.grunnlag.consumer.arbeidsforhold.ArbeidsforholdConsumer.Companion.LOGGER import no.nav.bidrag.grunnlag.consumer.arbeidsforhold.EnhetsregisterConsumer import no.nav.bidrag.grunnlag.consumer.arbeidsforhold.api.Arbeidsforhold import no.nav.bidrag.grunnlag.consumer.arbeidsforhold.api.Arbeidssted @@ -17,6 +18,7 @@ import no.nav.bidrag.transport.behandling.grunnlag.response.ArbeidsforholdGrunnl import no.nav.bidrag.transport.behandling.grunnlag.response.FeilrapporteringDto import no.nav.bidrag.transport.behandling.grunnlag.response.Permisjon import no.nav.bidrag.transport.behandling.grunnlag.response.Permittering +import org.springframework.http.HttpStatus class HentArbeidsforholdService( private val arbeidsforholdConsumer: ArbeidsforholdConsumer, @@ -148,53 +150,59 @@ class HentArbeidsforholdService( arbeidssted: Arbeidssted?, feilrapporteringListe: MutableList, ident: String, - ): Arbeidsgiverinfo? { - return if (arbeidssted?.type == "Underenhet") { - val orgnr = arbeidssted.identer?.filter { it.type == "ORGANISASJONSNUMMER" }?.get(0)?.ident - var navn: String? = null - - if (orgnr != null) { - when (val restResponseEnhetsregister = enhetsregisterConsumer.hentEnhetsinfo(HentEnhetsregisterRequest(orgnr))) { - is RestResponse.Success -> { - navn = restResponseEnhetsregister.body.navn?.navnelinje1 - } + ): Arbeidsgiverinfo? = if (arbeidssted?.type == "Underenhet") { + val orgnr = arbeidssted.identer?.filter { it.type == "ORGANISASJONSNUMMER" }?.get(0)?.ident + var navn: String? = null + + if (orgnr != null) { + when (val restResponseEnhetsregister = enhetsregisterConsumer.hentEnhetsinfo(HentEnhetsregisterRequest(orgnr))) { + is RestResponse.Success -> { + navn = restResponseEnhetsregister.body.navn?.navnelinje1 + } - is RestResponse.Failure -> { + is RestResponse.Failure -> { + if (restResponseEnhetsregister.statusCode == HttpStatus.NOT_FOUND) { SECURE_LOGGER.warn( + "Arbeidsgivernavn ikke funnet fra enhetsregisteret for orgnr $orgnr. " + + "Statuskode ${restResponseEnhetsregister.statusCode.value()}", + ) + } else { + LOGGER.error( "Feil ved henting av arbeidsgivernavn fra enhetsregisteret for orgnr $orgnr. " + "Statuskode ${restResponseEnhetsregister.statusCode.value()}", ) - feilrapporteringListe.add( - FeilrapporteringDto( - grunnlagstype = GrunnlagRequestType.ARBEIDSFORHOLD, - personId = ident, - periodeFra = null, - periodeTil = null, - feiltype = evaluerFeiltype( - melding = restResponseEnhetsregister.message, - httpStatuskode = restResponseEnhetsregister.statusCode, - ), - feilmelding = evaluerFeilmelding( - melding = restResponseEnhetsregister.message, - grunnlagstype = GrunnlagRequestType.ARBEIDSFORHOLD, - ), - ), + SECURE_LOGGER.error( + "Feil ved henting av arbeidsgivernavn fra enhetsregisteret for orgnr $orgnr. " + + "Statuskode ${restResponseEnhetsregister.statusCode.value()}", ) } + feilrapporteringListe.add( + FeilrapporteringDto( + grunnlagstype = GrunnlagRequestType.ARBEIDSFORHOLD, + personId = ident, + periodeFra = null, + periodeTil = null, + feiltype = evaluerFeiltype( + melding = restResponseEnhetsregister.message, + httpStatuskode = restResponseEnhetsregister.statusCode, + ), + feilmelding = evaluerFeilmelding( + melding = restResponseEnhetsregister.message, + grunnlagstype = GrunnlagRequestType.ARBEIDSFORHOLD, + ), + ), + ) } } - - Arbeidsgiverinfo( - orgnr = orgnr, - navn = navn, - ) - } else { - null } + + Arbeidsgiverinfo( + orgnr = orgnr, + navn = navn, + ) + } else { + null } } -data class Arbeidsgiverinfo( - val orgnr: String?, - val navn: String?, -) +data class Arbeidsgiverinfo(val orgnr: String?, val navn: String?) diff --git a/src/main/kotlin/no/nav/bidrag/grunnlag/service/HentBarnetilleggService.kt b/src/main/kotlin/no/nav/bidrag/grunnlag/service/HentBarnetilleggService.kt index ff20197f..ba932010 100644 --- a/src/main/kotlin/no/nav/bidrag/grunnlag/service/HentBarnetilleggService.kt +++ b/src/main/kotlin/no/nav/bidrag/grunnlag/service/HentBarnetilleggService.kt @@ -8,15 +8,15 @@ import no.nav.bidrag.grunnlag.consumer.pensjon.PensjonConsumer import no.nav.bidrag.grunnlag.consumer.pensjon.api.BarnetilleggPensjon import no.nav.bidrag.grunnlag.consumer.pensjon.api.HentBarnetilleggPensjonRequest import no.nav.bidrag.grunnlag.exception.RestResponse +import no.nav.bidrag.grunnlag.service.InntektskomponentenService.Companion.LOGGER import no.nav.bidrag.grunnlag.util.GrunnlagUtil.Companion.evaluerFeilmelding import no.nav.bidrag.grunnlag.util.GrunnlagUtil.Companion.evaluerFeiltype import no.nav.bidrag.grunnlag.util.GrunnlagUtil.Companion.tilJson import no.nav.bidrag.transport.behandling.grunnlag.response.BarnetilleggGrunnlagDto import no.nav.bidrag.transport.behandling.grunnlag.response.FeilrapporteringDto +import org.springframework.http.HttpStatus -class HentBarnetilleggService( - private val pensjonConsumer: PensjonConsumer, -) { +class HentBarnetilleggService(private val pensjonConsumer: PensjonConsumer) { fun hentBarnetilleggPensjon(barnetilleggPensjonRequestListe: List): HentGrunnlagGenericDto { val barnetilleggPensjonListe = mutableListOf() @@ -44,10 +44,21 @@ class HentBarnetilleggService( } is RestResponse.Failure -> { - SECURE_LOGGER.warn( - "Feil ved henting av barnetillegg pensjon for ${it.personId}. " + - "Statuskode ${restResponseBarnetillegg.statusCode.value()}", - ) + if (restResponseBarnetillegg.statusCode == HttpStatus.NOT_FOUND) { + SECURE_LOGGER.warn( + "Barnetillegg pensjon ikke funnet for ${it.personId}. " + + "Statuskode ${restResponseBarnetillegg.statusCode.value()}", + ) + } else { + LOGGER.error( + "Feil ved henting av barnetillegg fra pensjon. Statuskode ${restResponseBarnetillegg.statusCode.value()}", + ) + SECURE_LOGGER.error( + "Feil ved henting av barnetillegg pensjon for ${it.personId}. " + + "Statuskode ${restResponseBarnetillegg.statusCode.value()}", + ) + } + feilrapporteringListe.add( FeilrapporteringDto( grunnlagstype = GrunnlagRequestType.BARNETILLEGG, diff --git a/src/main/kotlin/no/nav/bidrag/grunnlag/service/HentBarnetilsynService.kt b/src/main/kotlin/no/nav/bidrag/grunnlag/service/HentBarnetilsynService.kt index 7851585f..a054376f 100644 --- a/src/main/kotlin/no/nav/bidrag/grunnlag/service/HentBarnetilsynService.kt +++ b/src/main/kotlin/no/nav/bidrag/grunnlag/service/HentBarnetilsynService.kt @@ -8,15 +8,15 @@ import no.nav.bidrag.grunnlag.consumer.familieefsak.FamilieEfSakConsumer import no.nav.bidrag.grunnlag.consumer.familieefsak.api.BarnetilsynRequest import no.nav.bidrag.grunnlag.consumer.familieefsak.api.BarnetilsynResponse import no.nav.bidrag.grunnlag.exception.RestResponse +import no.nav.bidrag.grunnlag.service.InntektskomponentenService.Companion.LOGGER import no.nav.bidrag.grunnlag.util.GrunnlagUtil.Companion.evaluerFeilmelding import no.nav.bidrag.grunnlag.util.GrunnlagUtil.Companion.evaluerFeiltype import no.nav.bidrag.grunnlag.util.GrunnlagUtil.Companion.tilJson import no.nav.bidrag.transport.behandling.grunnlag.response.BarnetilsynGrunnlagDto import no.nav.bidrag.transport.behandling.grunnlag.response.FeilrapporteringDto +import org.springframework.http.HttpStatus -class HentBarnetilsynService( - private val familieEfSakConsumer: FamilieEfSakConsumer, -) { +class HentBarnetilsynService(private val familieEfSakConsumer: FamilieEfSakConsumer) { fun hentBarnetilsyn(barnetilsynRequestListe: List): HentGrunnlagGenericDto { val barnetilsynListe = mutableListOf() @@ -37,10 +37,22 @@ class HentBarnetilsynService( } is RestResponse.Failure -> { - SECURE_LOGGER.warn( - "Feil ved henting av barnetilsyn for ${it.personId}. " + - "Statuskode ${restResponseBarnetilsyn.statusCode.value()}", - ) + if (restResponseBarnetilsyn.statusCode == HttpStatus.NOT_FOUND) { + SECURE_LOGGER.warn( + "Stønad til barnetilsyn fra Enslig Forsørger ikke funnet for ${it.personId}. " + + "Statuskode ${restResponseBarnetilsyn.statusCode.value()}", + ) + } else { + LOGGER.error( + "Feil ved henting av stønad til barnetilsyn fra Enslig Forsørger. " + + "Statuskode ${restResponseBarnetilsyn.statusCode.value()}", + ) + SECURE_LOGGER.error( + "Feil ved henting av stønad til barnetilsyn fra Enslig Forsørger for ${it.personId}. " + + "Statuskode ${restResponseBarnetilsyn.statusCode.value()}", + ) + } + feilrapporteringListe.add( FeilrapporteringDto( grunnlagstype = GrunnlagRequestType.BARNETILSYN, diff --git "a/src/main/kotlin/no/nav/bidrag/grunnlag/service/HentKontantst\303\270tteService.kt" "b/src/main/kotlin/no/nav/bidrag/grunnlag/service/HentKontantst\303\270tteService.kt" index cdf02b86..1c06f918 100644 --- "a/src/main/kotlin/no/nav/bidrag/grunnlag/service/HentKontantst\303\270tteService.kt" +++ "b/src/main/kotlin/no/nav/bidrag/grunnlag/service/HentKontantst\303\270tteService.kt" @@ -6,17 +6,17 @@ import no.nav.bidrag.grunnlag.consumer.familiekssak.FamilieKsSakConsumer import no.nav.bidrag.grunnlag.consumer.familiekssak.api.BisysDto import no.nav.bidrag.grunnlag.consumer.familiekssak.api.BisysResponsDto import no.nav.bidrag.grunnlag.exception.RestResponse +import no.nav.bidrag.grunnlag.service.InntektskomponentenService.Companion.LOGGER import no.nav.bidrag.grunnlag.util.GrunnlagUtil.Companion.evaluerFeilmelding import no.nav.bidrag.grunnlag.util.GrunnlagUtil.Companion.evaluerFeiltype import no.nav.bidrag.grunnlag.util.GrunnlagUtil.Companion.tilJson import no.nav.bidrag.transport.behandling.grunnlag.response.FeilrapporteringDto import no.nav.bidrag.transport.behandling.grunnlag.response.KontantstøtteGrunnlagDto +import org.springframework.http.HttpStatus import java.time.LocalDate import java.time.YearMonth -class HentKontantstøtteService( - private val familieKsSakConsumer: FamilieKsSakConsumer, -) { +class HentKontantstøtteService(private val familieKsSakConsumer: FamilieKsSakConsumer) { fun hentKontantstøtte( kontantstøtteRequestListe: List, @@ -46,10 +46,21 @@ class HentKontantstøtteService( } is RestResponse.Failure -> { - SECURE_LOGGER.warn( - "Feil ved henting av kontantstøtte for $personIdListe. " + - "Statuskode ${restResponseKontantstøtte.statusCode.value()}", - ) + if (restResponseKontantstøtte.statusCode == HttpStatus.NOT_FOUND) { + SECURE_LOGGER.warn( + "Kontantstøtte ikke funnet for $personIdListe. " + + "Statuskode ${restResponseKontantstøtte.statusCode.value()}", + ) + } else { + LOGGER.error( + "Feil ved henting av kontantstøtte fra KS-Sak. Statuskode ${restResponseKontantstøtte.statusCode.value()}", + ) + SECURE_LOGGER.error( + "Feil ved henting av kontantstøtte for $personIdListe. " + + "Statuskode ${restResponseKontantstøtte.statusCode.value()}", + ) + } + feilrapporteringListe.add( FeilrapporteringDto( grunnlagstype = GrunnlagRequestType.KONTANTSTØTTE, diff --git a/src/main/kotlin/no/nav/bidrag/grunnlag/service/HentRelatertePersonerService.kt b/src/main/kotlin/no/nav/bidrag/grunnlag/service/HentRelatertePersonerService.kt index 0e0103ad..93a21320 100644 --- a/src/main/kotlin/no/nav/bidrag/grunnlag/service/HentRelatertePersonerService.kt +++ b/src/main/kotlin/no/nav/bidrag/grunnlag/service/HentRelatertePersonerService.kt @@ -9,6 +9,7 @@ import no.nav.bidrag.grunnlag.bo.PersonBo import no.nav.bidrag.grunnlag.consumer.bidragperson.BidragPersonConsumer import no.nav.bidrag.grunnlag.consumer.bidragperson.api.HusstandsmedlemmerRequest import no.nav.bidrag.grunnlag.exception.RestResponse +import no.nav.bidrag.grunnlag.service.InntektskomponentenService.Companion.LOGGER import no.nav.bidrag.grunnlag.util.GrunnlagUtil.Companion.evaluerFeilmelding import no.nav.bidrag.grunnlag.util.GrunnlagUtil.Companion.evaluerFeiltype import no.nav.bidrag.grunnlag.util.GrunnlagUtil.Companion.tilJson @@ -20,6 +21,7 @@ import no.nav.bidrag.transport.person.ForelderBarnRelasjonDto import no.nav.bidrag.transport.person.Husstandsmedlem import no.nav.bidrag.transport.person.NavnFødselDødDto import no.nav.bidrag.transport.person.PersonRequest +import org.springframework.http.HttpStatus import java.time.LocalDate class HentRelatertePersonerService(private val bidragPersonConsumer: BidragPersonConsumer) { @@ -196,10 +198,21 @@ class HentRelatertePersonerService(private val bidragPersonConsumer: BidragPerso } is RestResponse.Failure -> { - SECURE_LOGGER.warn( - "Feil ved henting av husstandsmedlemmer for ${personIdOgPeriode.personId}. " + - "Statuskode ${restResponseHusstandsmedlemmer.statusCode.value()}", - ) + if (restResponseHusstandsmedlemmer.statusCode == HttpStatus.NOT_FOUND) { + SECURE_LOGGER.warn( + "Feil ved henting av husstandsmedlemmer for ${personIdOgPeriode.personId}. " + + "Statuskode ${restResponseHusstandsmedlemmer.statusCode.value()}", + ) + } else { + LOGGER.error( + "Feil ved henting av husstandsmedlemmer fra bidrag-person/PDL ${restResponseHusstandsmedlemmer.statusCode.value()}", + ) + SECURE_LOGGER.error( + "Feil ved henting av husstandsmedlemmer for ${personIdOgPeriode.personId}. " + + "Statuskode ${restResponseHusstandsmedlemmer.statusCode.value()}", + ) + } + feilrapporteringListe.add( FeilrapporteringDto( grunnlagstype = GrunnlagRequestType.HUSSTANDSMEDLEMMER_OG_EGNE_BARN, @@ -240,10 +253,21 @@ class HentRelatertePersonerService(private val bidragPersonConsumer: BidragPerso } is RestResponse.Failure -> { - SECURE_LOGGER.warn( - "Feil ved henting av forelder-barn-relasjoner for ${personident.verdi}. " + - "Statuskode ${restResponseForelderBarnRelasjon.statusCode.value()}", - ) + if (restResponseForelderBarnRelasjon.statusCode == HttpStatus.NOT_FOUND) { + SECURE_LOGGER.warn( + "Feil ved henting av forelder-barn-relasjoner for ${personident.verdi}. " + + "Statuskode ${restResponseForelderBarnRelasjon.statusCode.value()}", + ) + } else { + LOGGER.error( + "Feil ved henting av forelder-barn-relasjon fra bidrag-person/PDL. " + + "Statuskode ${restResponseForelderBarnRelasjon.statusCode.value()}", + ) + SECURE_LOGGER.error( + "Feil ved henting av forelder-barn-relasjoner for ${personident.verdi}. " + + "Statuskode ${restResponseForelderBarnRelasjon.statusCode.value()}", + ) + } feilrapporteringListe.add( FeilrapporteringDto( grunnlagstype = GrunnlagRequestType.HUSSTANDSMEDLEMMER_OG_EGNE_BARN, @@ -314,10 +338,20 @@ class HentRelatertePersonerService(private val bidragPersonConsumer: BidragPerso } is RestResponse.Failure -> { - SECURE_LOGGER.warn( - "Feil ved henting av navn og fødselsdato for ${personident.verdi}. " + - "Statuskode ${restResponseFoedselOgDoed.statusCode.value()}", - ) + if (restResponseFoedselOgDoed.statusCode == HttpStatus.NOT_FOUND) { + SECURE_LOGGER.warn( + "Feil ved henting av navn og fødselsdato for ${personident.verdi}. " + + "Statuskode ${restResponseFoedselOgDoed.statusCode.value()}", + ) + } else { + LOGGER.error( + "Feil ved henting av navn og fødselsdato fra bidrag-person/PDL. Statuskode ${restResponseFoedselOgDoed.statusCode.value()}", + ) + SECURE_LOGGER.error( + "Feil ved henting av navn og fødselsdato for ${personident.verdi}. " + + "Statuskode ${restResponseFoedselOgDoed.statusCode.value()}", + ) + } feilrapporteringListe.add( FeilrapporteringDto( grunnlagstype = GrunnlagRequestType.HUSSTANDSMEDLEMMER_OG_EGNE_BARN, @@ -412,9 +446,18 @@ class HentRelatertePersonerService(private val bidragPersonConsumer: BidragPerso } is RestResponse.Failure -> { - SECURE_LOGGER.warn( - "Feil ved henting av ektefelles personident for $personId. Statuskode ${restResponseSivilstand.statusCode.value()}", - ) + if (restResponseSivilstand.statusCode == HttpStatus.NOT_FOUND) { + SECURE_LOGGER.warn( + "Feil ved henting av ektefelles personident for $personId. Statuskode ${restResponseSivilstand.statusCode.value()}", + ) + } else { + LOGGER.error( + "Feil ved henting av ektefelles personident fra bidrag-person/PDL. Statuskode ${restResponseSivilstand.statusCode.value()}", + ) + SECURE_LOGGER.error( + "Feil ved henting av ektefelles personident for $personId. Statuskode ${restResponseSivilstand.statusCode.value()}", + ) + } return emptyList() } } diff --git a/src/main/kotlin/no/nav/bidrag/grunnlag/service/HentSivilstandService.kt b/src/main/kotlin/no/nav/bidrag/grunnlag/service/HentSivilstandService.kt index cef28914..39c622d8 100644 --- a/src/main/kotlin/no/nav/bidrag/grunnlag/service/HentSivilstandService.kt +++ b/src/main/kotlin/no/nav/bidrag/grunnlag/service/HentSivilstandService.kt @@ -5,12 +5,14 @@ import no.nav.bidrag.domene.ident.Personident import no.nav.bidrag.grunnlag.SECURE_LOGGER import no.nav.bidrag.grunnlag.consumer.bidragperson.BidragPersonConsumer import no.nav.bidrag.grunnlag.exception.RestResponse +import no.nav.bidrag.grunnlag.service.InntektskomponentenService.Companion.LOGGER import no.nav.bidrag.grunnlag.util.GrunnlagUtil.Companion.evaluerFeilmelding import no.nav.bidrag.grunnlag.util.GrunnlagUtil.Companion.evaluerFeiltype import no.nav.bidrag.grunnlag.util.GrunnlagUtil.Companion.tilJson import no.nav.bidrag.transport.behandling.grunnlag.response.FeilrapporteringDto import no.nav.bidrag.transport.behandling.grunnlag.response.SivilstandGrunnlagDto import no.nav.bidrag.transport.person.SivilstandPdlHistorikkDto +import org.springframework.http.HttpStatus class HentSivilstandService(private val bidragPersonConsumer: BidragPersonConsumer) { @@ -30,7 +32,17 @@ class HentSivilstandService(private val bidragPersonConsumer: BidragPersonConsum } is RestResponse.Failure -> { - SECURE_LOGGER.warn("Feil ved henting av sivilstand for ${it.personId}. Statuskode ${restResponseSivilstand.statusCode.value()}") + if (restResponseSivilstand.statusCode == HttpStatus.NOT_FOUND) { + SECURE_LOGGER.warn("Sivilstand ikke funnet for ${it.personId}. Statuskode ${restResponseSivilstand.statusCode.value()}") + } else { + LOGGER.error( + "Feil ved henting av sivilstanda bidrag-person/PDL. Statuskode ${restResponseSivilstand.statusCode.value()}", + ) + SECURE_LOGGER.error( + "Feil ved henting av sivilstand for ${it.personId}. Statuskode ${restResponseSivilstand.statusCode.value()}", + ) + } + feilrapporteringListe.add( FeilrapporteringDto( grunnlagstype = GrunnlagRequestType.SIVILSTAND, diff --git a/src/main/kotlin/no/nav/bidrag/grunnlag/service/HentSkattegrunnlagService.kt b/src/main/kotlin/no/nav/bidrag/grunnlag/service/HentSkattegrunnlagService.kt index c42068b7..2f70a77f 100644 --- a/src/main/kotlin/no/nav/bidrag/grunnlag/service/HentSkattegrunnlagService.kt +++ b/src/main/kotlin/no/nav/bidrag/grunnlag/service/HentSkattegrunnlagService.kt @@ -7,6 +7,7 @@ import no.nav.bidrag.grunnlag.consumer.skattegrunnlag.SigrunConsumer import no.nav.bidrag.grunnlag.consumer.skattegrunnlag.api.HentSummertSkattegrunnlagRequest import no.nav.bidrag.grunnlag.consumer.skattegrunnlag.api.HentSummertSkattegrunnlagResponse import no.nav.bidrag.grunnlag.exception.RestResponse +import no.nav.bidrag.grunnlag.service.InntektskomponentenService.Companion.LOGGER import no.nav.bidrag.grunnlag.util.GrunnlagUtil.Companion.erBnrEllerNpid import no.nav.bidrag.grunnlag.util.GrunnlagUtil.Companion.evaluerFeilmelding import no.nav.bidrag.grunnlag.util.GrunnlagUtil.Companion.evaluerFeiltype @@ -18,9 +19,7 @@ import org.springframework.http.HttpStatus import java.math.BigDecimal import java.time.LocalDate -class HentSkattegrunnlagService( - private val sigrunConsumer: SigrunConsumer, -) { +class HentSkattegrunnlagService(private val sigrunConsumer: SigrunConsumer) { companion object { const val INNTEKTSAAR_IKKE_STØTTET = "Oppgitt inntektsår er ikke støttet" const val FANT_IKKE_SKATTEGRUNNLAG_PROD = "Fant ikke summert skattegrunnlag" @@ -70,7 +69,7 @@ class HentSkattegrunnlagService( if ((restResponseSkattegrunnlag.statusCode == HttpStatus.NOT_FOUND) && (inntektsårIkkeStøttet(restResponseSkattegrunnlag.message)) ) { - SECURE_LOGGER.warn("Skattegrunnlag er ikke tilgjengelig enda for ${it.personId} og år $inntektÅr") + SECURE_LOGGER.warn("Skattegrunnlag er ikke tilgjengelig ennå for ${it.personId} og år $inntektÅr") // Legger ut tom liste hvis det ikke finnes data } else if ((restResponseSkattegrunnlag.statusCode == HttpStatus.NOT_FOUND) && @@ -88,7 +87,10 @@ class HentSkattegrunnlagService( ), ) } else { - SECURE_LOGGER.warn( + LOGGER.error( + "Feil ved henting av skattegrunnlag. Statuskode ${restResponseSkattegrunnlag.statusCode.value()}", + ) + SECURE_LOGGER.error( "Feil ved henting av skattegrunnlag for ${it.personId} og år $inntektÅr. " + "Statuskode ${restResponseSkattegrunnlag.statusCode.value()}", ) diff --git "a/src/main/kotlin/no/nav/bidrag/grunnlag/service/HentUtvidetBarnetrygdOgSm\303\245barnstilleggService.kt" "b/src/main/kotlin/no/nav/bidrag/grunnlag/service/HentUtvidetBarnetrygdOgSm\303\245barnstilleggService.kt" index 411aff01..e07b8dae 100644 --- "a/src/main/kotlin/no/nav/bidrag/grunnlag/service/HentUtvidetBarnetrygdOgSm\303\245barnstilleggService.kt" +++ "b/src/main/kotlin/no/nav/bidrag/grunnlag/service/HentUtvidetBarnetrygdOgSm\303\245barnstilleggService.kt" @@ -10,12 +10,11 @@ import no.nav.bidrag.grunnlag.util.GrunnlagUtil.Companion.evaluerFeilmelding import no.nav.bidrag.grunnlag.util.GrunnlagUtil.Companion.evaluerFeiltype import no.nav.bidrag.grunnlag.util.GrunnlagUtil.Companion.tilJson import no.nav.bidrag.transport.behandling.grunnlag.response.FeilrapporteringDto +import org.springframework.http.HttpStatus import java.math.BigDecimal import java.time.LocalDate -class HentUtvidetBarnetrygdOgSmåbarnstilleggService( - private val familieBaSakConsumer: FamilieBaSakConsumer, -) { +class HentUtvidetBarnetrygdOgSmåbarnstilleggService(private val familieBaSakConsumer: FamilieBaSakConsumer) { fun hentUbst(ubstRequestListe: List): HentGrunnlagGenericDto { val ubstListe = mutableListOf() @@ -38,10 +37,17 @@ class HentUtvidetBarnetrygdOgSmåbarnstilleggService( } is RestResponse.Failure -> { - SECURE_LOGGER.warn( - "Feil ved henting av utvidet barnetrygd og småbarnstillegg for ${it.personId}. " + - "Statuskode ${restResponseUbst.statusCode.value()}", - ) + if (restResponseUbst.statusCode == HttpStatus.NOT_FOUND) { + SECURE_LOGGER.warn( + "Feil ved henting av utvidet barnetrygd og småbarnstillegg for ${it.personId}. " + + "Statuskode ${restResponseUbst.statusCode.value()}", + ) + } else { + SECURE_LOGGER.error( + "Feil ved henting av utvidet barnetrygd og småbarnstillegg for ${it.personId}. " + + "Statuskode ${restResponseUbst.statusCode.value()}", + ) + } feilrapporteringListe.add( FeilrapporteringDto( grunnlagstype = GrunnlagRequestType.UTVIDET_BARNETRYGD_OG_SMÅBARNSTILLEGG, diff --git a/src/main/kotlin/no/nav/bidrag/grunnlag/service/InntektskomponentenService.kt b/src/main/kotlin/no/nav/bidrag/grunnlag/service/InntektskomponentenService.kt index 43bdac7c..ae657b89 100644 --- a/src/main/kotlin/no/nav/bidrag/grunnlag/service/InntektskomponentenService.kt +++ b/src/main/kotlin/no/nav/bidrag/grunnlag/service/InntektskomponentenService.kt @@ -23,9 +23,7 @@ import org.springframework.http.HttpStatusCode import org.springframework.stereotype.Service @Service -class InntektskomponentenService( - private val inntektskomponentenConsumer: InntektskomponentenConsumer, -) { +class InntektskomponentenService(private val inntektskomponentenConsumer: InntektskomponentenConsumer) { companion object { @JvmStatic val LOGGER: Logger = LoggerFactory.getLogger(InntektskomponentenService::class.java) @@ -55,10 +53,21 @@ class InntektskomponentenService( } is RestResponse.Failure -> { - SECURE_LOGGER.warn( - "Feil ved henting av inntekter med abonnement for ${inntektListeRequest.ident.identifikator} for perioden " + - "${inntektListeRequest.maanedFom} - ${inntektListeRequest.maanedTom}. Prøver å hente inntekter uten abonnement.", - ) + if (restResponseInntekt.statusCode == HttpStatus.NOT_FOUND) { + SECURE_LOGGER.warn( + "Feil ved henting av inntekter med abonnement for ${inntektListeRequest.ident.identifikator} for perioden " + + "${inntektListeRequest.maanedFom} - ${inntektListeRequest.maanedTom}. Prøver å hente inntekter uten abonnement.", + ) + } else { + LOGGER.error( + "Feil ved henting av inntekter med abonnement fra Inntektskomponenten. " + + "Statuskode ${restResponseInntekt.statusCode.value()}", + ) + SECURE_LOGGER.error( + "Feil ved henting av inntekter med abonnement for ${inntektListeRequest.ident.identifikator} for perioden " + + "${inntektListeRequest.maanedFom} - ${inntektListeRequest.maanedTom}. Prøver å hente inntekter uten abonnement.", + ) + } // Respons ikke OK. Gjør nytt forsøk, med kall mot hentInntektListe try { when (val restResponse2Inntekt = inntektskomponentenConsumer.hentInntekter(inntektListeRequest, false)) { @@ -76,11 +85,23 @@ class InntektskomponentenService( } is RestResponse.Failure -> { - SECURE_LOGGER.warn( - "Feil ved henting av inntekter uten abonnement for ${inntektListeRequest.ident.identifikator} for perioden " + - "${inntektListeRequest.maanedFom} - ${inntektListeRequest.maanedTom}. " + - "Statuskode ${restResponse2Inntekt.statusCode.value()}", - ) + if (restResponseInntekt.statusCode == HttpStatus.NOT_FOUND) { + SECURE_LOGGER.warn( + "Feil ved henting av inntekter uten abonnement for ${inntektListeRequest.ident.identifikator} for perioden " + + "${inntektListeRequest.maanedFom} - ${inntektListeRequest.maanedTom}. " + + "Statuskode ${restResponse2Inntekt.statusCode.value()}", + ) + } else { + LOGGER.error( + "Feil ved henting av inntekter uten abonnement fra Inntektskomponenten. " + + "Statuskode ${restResponseInntekt.statusCode.value()}", + ) + SECURE_LOGGER.error( + "Feil ved henting av inntekter uten abonnement for ${inntektListeRequest.ident.identifikator} for perioden " + + "${inntektListeRequest.maanedFom} - ${inntektListeRequest.maanedTom}. " + + "Statuskode ${restResponse2Inntekt.statusCode.value()}", + ) + } httpStatus = restResponse2Inntekt.statusCode melding = restResponse2Inntekt.message ?: "" }