Skip to content

Commit

Permalink
🔊 Fjern KRR-logging og logg at det gjøres oppslag på personen
Browse files Browse the repository at this point in the history
Logger oppslag for en enkel måte å få tak i en trace id på hvis vi har
behov for det ved feilsøking
  • Loading branch information
chsko committed Jun 28, 2024
1 parent 69e55e1 commit 01ebbc0
Show file tree
Hide file tree
Showing 2 changed files with 20 additions and 13 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -66,12 +66,16 @@ class PersonQuery(
): DataFetcherResult<Person?> {
if (fnr == null) {
if (aktorId == null) {
return DataFetcherResult.newResult<Person?>()
.error(getBadRequestError("Requesten mangler både fødselsnummer og aktørId")).build()
return DataFetcherResult
.newResult<Person?>()
.error(getBadRequestError("Requesten mangler både fødselsnummer og aktørId"))
.build()
}
if (aktorId.length != 13) {
return DataFetcherResult.newResult<Person?>()
.error(getBadRequestError("Feil lengde på parameter aktorId: ${aktorId.length}")).build()
return DataFetcherResult
.newResult<Person?>()
.error(getBadRequestError("Feil lengde på parameter aktorId: ${aktorId.length}"))
.build()
}
}

Expand All @@ -85,7 +89,9 @@ class PersonQuery(
} catch (e: Exception) {
val fødselsnumre = personApiDao.finnFødselsnumre(aktorId.toLong()).toSet()
auditLog(env.graphQlContext, aktorId, null, getFlereFødselsnumreError(fødselsnumre).message)
return DataFetcherResult.newResult<Person?>().error(getFlereFødselsnumreError(fødselsnumre))
return DataFetcherResult
.newResult<Person?>()
.error(getFlereFødselsnumreError(fødselsnumre))
.build()
}
}
Expand All @@ -95,6 +101,7 @@ class PersonQuery(
auditLog(env.graphQlContext, fnr ?: aktorId!!, null, getNotFoundError(fnr).message)
return DataFetcherResult.newResult<Person?>().error(getNotFoundError(fnr)).build()
}
sikkerLogg.info("Personppslag på fnr=$fødselsnummer")

if (!personApiDao.spesialistHarPersonKlarForVisningISpeil(fødselsnummer)) {
auditLog(env.graphQlContext, fødselsnummer, false, null)
Expand Down Expand Up @@ -167,7 +174,8 @@ class PersonQuery(
}

private fun getFlereFødselsnumreError(fødselsnumre: Set<String>): GraphQLError =
GraphqlErrorException.newErrorException()
GraphqlErrorException
.newErrorException()
.message("Mer enn ett fødselsnummer for personen")
.extensions(
mapOf(
Expand All @@ -178,15 +186,16 @@ class PersonQuery(
).build()

private fun getSnapshotValidationError(): GraphQLError =
GraphqlErrorException.newErrorException()
GraphqlErrorException
.newErrorException()
.message(
"Lagret snapshot stemmer ikke overens med forventet format. Dette kommer som regel av at noen har gjort endringer på formatet men glemt å bumpe versjonsnummeret.",
)
.extensions(mapOf("code" to 501, "field" to "person"))
).extensions(mapOf("code" to 501, "field" to "person"))
.build()

private fun getBadRequestError(melding: String): GraphQLError =
GraphqlErrorException.newErrorException()
GraphqlErrorException
.newErrorException()
.message(melding)
.extensions(mapOf("code" to 400))
.build()
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,6 @@ class KRRClient(
val accessToken = accessTokenClient.hentAccessToken(scope)
val callId = UUID.randomUUID().toString()

sikkerLogg.info("Henter reservasjonsstatus for $fnr")
return httpClient
.get("$apiUrl/rest/v1/person") {
header("Authorization", "Bearer $accessToken")
Expand All @@ -53,8 +52,7 @@ class KRRClient(
logg.error("Feil under kall til Kontakt- og reservasjonsregisteret")
sikkerLogg.error("Feil under kall til Kontakt- og reservasjonsregisteret", e)
} finally {
val tidBrukt = timer.observeDuration()
sikkerLogg.info("Kall til KRR har blitt forsøkt hentet for $fnr, kallet tok $tidBrukt sekunder")
timer.observeDuration()
}

return null
Expand Down

0 comments on commit 01ebbc0

Please sign in to comment.