Legg på logging kontekst og logging av body til sikkerlogg ved feil

Author: androa

dp-inntekt-api/src/main/kotlin/no/nav/dagpenger/inntekt/mapping/MapFromGUIInntekt.kt

@@ -1,5 +1,6 @@
 package no.nav.dagpenger.inntekt.mapping
 
+import mu.KotlinLogging
 import no.nav.dagpenger.inntekt.db.DetachedInntekt
 import no.nav.dagpenger.inntekt.db.StoredInntekt
 import no.nav.dagpenger.inntekt.inntektskomponenten.v1.ArbeidsInntektInformasjon
@@ -8,7 +9,9 @@ import no.nav.dagpenger.inntekt.inntektskomponenten.v1.Inntekt
 import no.nav.dagpenger.inntekt.inntektskomponenten.v1.InntektkomponentResponse
 import no.nav.dagpenger.inntekt.inntektskomponenten.v1.TilleggInformasjon
 import no.nav.dagpenger.inntekt.inntektskomponenten.v1.TilleggInformasjonsDetaljer
-import java.lang.IllegalArgumentException
+
+private val logg = KotlinLogging.logger {}
+private val sikkerlogg = KotlinLogging.logger("tjenestekall.MapFromGUIInntekt")
 
 fun mapToStoredInntekt(guiInntekt: GUIInntekt): StoredInntekt = guiInntekt.inntektId?.let {
     StoredInntekt(
@@ -20,7 +23,10 @@ fun mapToStoredInntekt(guiInntekt: GUIInntekt): StoredInntekt = guiInntekt.innte
         ),
         guiInntekt.manueltRedigert
     )
-} ?: throw IllegalArgumentException("missing innktektId")
+} ?: throw IllegalArgumentException("missing innktektId").also {
+    logg.error(it) { "Mangler inntektId" }
+    sikkerlogg.error(it) { "Mangler inntektId. guiInntekt=$guiInntekt" }
+}
 
 fun mapToDetachedInntekt(guiInntekt: GUIInntekt): DetachedInntekt =
     DetachedInntekt(
@@ -60,7 +66,15 @@ private fun mapToArbeidsInntektMaaneder(arbeidsMaaneder: List<GUIArbeidsInntektM
                         inntekt.utloeserArbeidsgiveravgift,
                         inntekt.informasjonsstatus,
                         datagrunnlagForVerdikode.type,
-                        datagrunnlagForVerdikode.forhold?.let { TilleggInformasjon(inntekt.tilleggsinformasjon?.kategori, TilleggInformasjonsDetaljer(inntekt.tilleggsinformasjon?.tilleggsinformasjonDetaljer?.detaljerType, it)) }
+                        datagrunnlagForVerdikode.forhold?.let {
+                            TilleggInformasjon(
+                                inntekt.tilleggsinformasjon?.kategori,
+                                TilleggInformasjonsDetaljer(
+                                    inntekt.tilleggsinformasjon?.tilleggsinformasjonDetaljer?.detaljerType,
+                                    it
+                                )
+                            )
+                        }
                     )
                 } ?: emptyList()
             )

dp-inntekt-api/src/main/kotlin/no/nav/dagpenger/inntekt/v1/UklassifisertInntektRoute.kt

@@ -62,7 +62,7 @@ fun Route.uklassifisertInntekt(
         route("/uklassifisert/{aktørId}/{kontekstType}/{kontekstId}/{beregningsDato}") {
             get {
                 withContext(Dispatchers.IO) {
-                    parseUrlPathParameters().run {
+                    withInntektRequest {
                         inntektStore.getInntektId(
                             Inntektparametre(
                                 aktørId = this.aktørId,
@@ -83,7 +83,7 @@ fun Route.uklassifisertInntekt(
             }
             post {
                 withContext(Dispatchers.IO) {
-                    parseUrlPathParameters().run {
+                    withInntektRequest {
                         val guiInntekt = call.receive<GUIInntekt>()
                         mapToStoredInntekt(guiInntekt).let {
                             inntektStore.storeInntekt(
@@ -121,7 +121,7 @@ fun Route.uklassifisertInntekt(
             get {
                 val callId = call.callId
                 withContext(Dispatchers.IO) {
-                    parseUrlPathParameters().run {
+                    withInntektRequest {
                         withLoggingContext(
                             "route" to "/uklassifisert/uncached",
                             "aktørId" to aktørId,
@@ -151,7 +151,7 @@ fun Route.uklassifisertInntekt(
 
             post {
                 withContext(Dispatchers.IO) {
-                    parseUrlPathParameters().run {
+                    withInntektRequest {
                         val guiInntekt = call.receive<GUIInntekt>()
                         mapToDetachedInntekt(guiInntekt).let {
                             inntektStore.storeInntekt(
@@ -209,14 +209,23 @@ private fun PipelineContext<Unit, ApplicationCall>.getSubject(): String {
     }
 }
 
-private fun PipelineContext<Unit, ApplicationCall>.parseUrlPathParameters(): InntektRequest = runCatching {
-    InntektRequest(
-        aktørId = call.parameters["aktørId"]!!,
-        kontekstId = call.parameters["kontekstId"]!!,
-        kontekstType = call.parameters["kontekstType"]!!,
-        beregningsDato = LocalDate.parse(call.parameters["beregningsDato"]!!)
-    )
-}.getOrElse { t -> throw IllegalArgumentException("Failed to parse parameters", t) }
+private inline fun PipelineContext<Unit, ApplicationCall>.withInntektRequest(block: InntektRequest.() -> Unit) {
+    val inntektRequest = runCatching {
+        InntektRequest(
+            aktørId = call.parameters["aktørId"]!!,
+            kontekstId = call.parameters["kontekstId"]!!,
+            kontekstType = call.parameters["kontekstType"]!!,
+            beregningsDato = LocalDate.parse(call.parameters["beregningsDato"]!!)
+        )
+    }.getOrElse { t -> throw IllegalArgumentException("Failed to parse parameters", t) }
+
+    withLoggingContext(
+        "kontekstId" to inntektRequest.kontekstId,
+        "kontekstType" to inntektRequest.kontekstType
+    ) {
+        block(inntektRequest)
+    }
+}
 
 data class InntektRequest(
     val aktørId: String,