Skip to content

Commit 0205125

Browse files
hestadhestad
committed
Bruk felles sikkerlogg
Forsikre oss om at vi logger til samme sikkerlogg på tvers av vedtak og enklere bruke sikkerlogg
1 parent 984d42e commit 0205125

33 files changed

+93
-125
lines changed

‎app/src/main/kotlin/no/nav/tiltakspenger/vedtak/App.kt

+2-2
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@ import arrow.core.right
55
import io.ktor.server.engine.embeddedServer
66
import io.ktor.server.netty.Netty
77
import mu.KotlinLogging
8+
import no.nav.tiltakspenger.felles.sikkerlogg
89
import no.nav.tiltakspenger.libs.jobber.LeaderPodLookup
910
import no.nav.tiltakspenger.libs.jobber.LeaderPodLookupClient
1011
import no.nav.tiltakspenger.libs.jobber.LeaderPodLookupFeil
@@ -22,11 +23,10 @@ fun main() {
2223
System.setProperty("logback.configurationFile", Configuration.logbackConfigurationFile())
2324

2425
val log = KotlinLogging.logger {}
25-
val securelog = KotlinLogging.logger("tjenestekall")
2626
log.info { "starting server" }
2727
Thread.setDefaultUncaughtExceptionHandler { _, e ->
2828
log.error { e }
29-
securelog.error(e) { e.message }
29+
sikkerlogg.error(e) { e.message }
3030
}
3131

3232
val dataSource = DataSourceSetup.createDatasource()

‎app/src/main/kotlin/no/nav/tiltakspenger/vedtak/auditlog/AuditService.kt

+12-8
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ package no.nav.tiltakspenger.vedtak.auditlog
22

33
import arrow.core.Either
44
import mu.KotlinLogging
5+
import no.nav.tiltakspenger.felles.sikkerlogg
56
import no.nav.tiltakspenger.libs.common.BehandlingId
67
import no.nav.tiltakspenger.libs.common.Fnr
78
import no.nav.tiltakspenger.libs.common.MeldekortId
@@ -12,7 +13,7 @@ import no.nav.tiltakspenger.saksbehandling.domene.sak.Saksnummer
1213
import java.lang.String.join
1314
import java.util.UUID
1415

15-
private val LOG = KotlinLogging.logger {}
16+
private val logg = KotlinLogging.logger {}
1617

1718
data class AuditLogEvent(
1819
val navIdent: String,
@@ -216,7 +217,7 @@ class AuditService(
216217
logLevel = AuditLogEvent.Level.INFO,
217218
),
218219
)
219-
}.onLeft { LOG.error { "Det oppstod en feil ved auditlogging" } }
220+
}.onLeft { logg.error { "Det oppstod en feil ved auditlogging" } }
220221
}
221222

222223
fun logMedMeldekortId(
@@ -241,7 +242,7 @@ class AuditService(
241242
logLevel = AuditLogEvent.Level.INFO,
242243
),
243244
)
244-
}.onLeft { LOG.error { "Det oppstod en feil ved auditlogging" } }
245+
}.onLeft { logg.error { "Det oppstod en feil ved auditlogging" } }
245246
}
246247

247248
fun logMedSakId(
@@ -266,7 +267,7 @@ class AuditService(
266267
logLevel = AuditLogEvent.Level.INFO,
267268
),
268269
)
269-
}.onLeft { LOG.error { "Det oppstod en feil ved auditlogging" } }
270+
}.onLeft { logg.error { "Det oppstod en feil ved auditlogging" } }
270271
}
271272

272273
fun logMedSaksnummer(
@@ -291,7 +292,7 @@ class AuditService(
291292
logLevel = AuditLogEvent.Level.INFO,
292293
),
293294
)
294-
}.onLeft { LOG.error { "Det oppstod en feil ved auditlogging" } }
295+
}.onLeft { logg.error { "Det oppstod en feil ved auditlogging" } }
295296
}
296297

297298
fun logMedBrukerId(
@@ -314,7 +315,7 @@ class AuditService(
314315
logLevel = AuditLogEvent.Level.INFO,
315316
),
316317
)
317-
}.onLeft { LOG.error { "Det oppstod en feil ved auditlogging" } }
318+
}.onLeft { logg.error { "Det oppstod en feil ved auditlogging" } }
318319
}
319320

320321
fun logForSøknadId(
@@ -339,7 +340,7 @@ class AuditService(
339340
logLevel = AuditLogEvent.Level.INFO,
340341
),
341342
)
342-
}.onLeft { LOG.error { "Det oppstod en feil ved auditlogging" } }
343+
}.onLeft { logg.error { "Det oppstod en feil ved auditlogging" } }
343344
}
344345

345346
fun logMedVedtakId(
@@ -364,6 +365,9 @@ class AuditService(
364365
logLevel = AuditLogEvent.Level.INFO,
365366
),
366367
)
367-
}.onLeft { LOG.error { "Det oppstod en feil ved auditlogging" } }
368+
}.onLeft {
369+
logg.error { "Det oppstod en feil ved auditlogging. Se sikkerlogg for mer exception." }
370+
sikkerlogg.error(it) { "Det oppstod en feil ved auditlogging" }
371+
}
368372
}
369373
}

‎app/src/main/kotlin/no/nav/tiltakspenger/vedtak/clients/DefaultHttpClient.kt

+2-2
Original file line numberDiff line numberDiff line change
@@ -19,10 +19,10 @@ import io.ktor.client.plugins.logging.Logging
1919
import io.ktor.http.ContentType
2020
import io.ktor.serialization.jackson.JacksonConverter
2121
import mu.KotlinLogging
22+
import no.nav.tiltakspenger.felles.sikkerlogg
2223
import java.time.Duration
2324

2425
private val LOG = KotlinLogging.logger {}
25-
private val SECURELOG = KotlinLogging.logger("tjenestekall")
2626
private const val SIXTY_SECONDS = 60L
2727

2828
// engine skal brukes primært i test-øyemed, når man sender med MockEngine.
@@ -59,7 +59,7 @@ private fun defaultSetup(objectMapper: ObjectMapper): HttpClientConfig<*>.() ->
5959
object : Logger {
6060
override fun log(message: String) {
6161
LOG.info("HttpClient detaljer logget til securelog")
62-
SECURELOG.info(message)
62+
sikkerlogg.info(message)
6363
}
6464
}
6565
level = LogLevel.ALL

‎app/src/main/kotlin/no/nav/tiltakspenger/vedtak/clients/KonfigurertHttpClient.kt

+2-2
Original file line numberDiff line numberDiff line change
@@ -16,10 +16,10 @@ import io.ktor.client.plugins.logging.Logger
1616
import io.ktor.client.plugins.logging.Logging
1717
import io.ktor.serialization.jackson.jackson
1818
import mu.KotlinLogging
19+
import no.nav.tiltakspenger.felles.sikkerlogg
1920
import java.time.Duration
2021

2122
private val LOG = KotlinLogging.logger {}
22-
private val SECURELOG = KotlinLogging.logger("tjenestekall")
2323

2424
private const val SIXTY_SECONDS = 60L
2525

@@ -66,7 +66,7 @@ private fun HttpClient.config(timeout: Long) =
6666
object : Logger {
6767
override fun log(message: String) {
6868
LOG.info("HttpClient detaljer logget til securelog")
69-
SECURELOG.info(message)
69+
sikkerlogg.info(message)
7070
}
7171
}
7272
level = LogLevel.INFO

‎app/src/main/kotlin/no/nav/tiltakspenger/vedtak/jobber/TaskExecutor.kt

+2-1
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@ import kotlinx.coroutines.CoroutineScope
55
import kotlinx.coroutines.Dispatchers
66
import kotlinx.coroutines.launch
77
import mu.KotlinLogging
8+
import no.nav.tiltakspenger.felles.sikkerlogg
89
import no.nav.tiltakspenger.libs.common.CorrelationId
910
import no.nav.tiltakspenger.libs.jobber.RunCheckFactory
1011
import no.nav.tiltakspenger.libs.jobber.StoppableJob
@@ -35,7 +36,7 @@ internal class TaskExecutor(
3536
initialDelay = initialDelay.toJavaDuration(),
3637
intervall = intervall.toJavaDuration(),
3738
logger = logger,
38-
sikkerLogg = KotlinLogging.logger("tjenestekall"),
39+
sikkerLogg = sikkerlogg,
3940
// Ref callIdMdc("call-id") i VedtakApi.kt
4041
mdcCallIdKey = "call-id",
4142
runJobCheck = listOf(runCheckFactory.leaderPod()),

‎app/src/main/kotlin/no/nav/tiltakspenger/vedtak/repository/behandling/BehandlingPostgresRepo.kt

+3-5
Original file line numberDiff line numberDiff line change
@@ -6,9 +6,9 @@ import kotliquery.Row
66
import kotliquery.Session
77
import kotliquery.TransactionalSession
88
import kotliquery.queryOf
9-
import mu.KotlinLogging
109
import no.nav.tiltakspenger.felles.exceptions.IkkeFunnetException
1110
import no.nav.tiltakspenger.felles.nå
11+
import no.nav.tiltakspenger.felles.sikkerlogg
1212
import no.nav.tiltakspenger.libs.common.BehandlingId
1313
import no.nav.tiltakspenger.libs.common.Fnr
1414
import no.nav.tiltakspenger.libs.common.SakId
@@ -31,8 +31,6 @@ import no.nav.tiltakspenger.vedtak.repository.søknad.SøknadDAO
3131
import org.intellij.lang.annotations.Language
3232
import java.time.LocalDateTime
3333

34-
private val SECURELOG = KotlinLogging.logger("tjenestekall")
35-
3634
class BehandlingPostgresRepo(
3735
private val sessionFactory: PostgresSessionFactory,
3836
) : BehandlingRepo {
@@ -151,7 +149,7 @@ class BehandlingPostgresRepo(
151149
behandling: Behandling,
152150
session: Session,
153151
) {
154-
SECURELOG.info { "Oppdaterer behandling ${behandling.id}" }
152+
sikkerlogg.info { "Oppdaterer behandling ${behandling.id}" }
155153

156154
val antRaderOppdatert =
157155
session.run(
@@ -182,7 +180,7 @@ class BehandlingPostgresRepo(
182180
behandling: Behandling,
183181
session: Session,
184182
) {
185-
SECURELOG.info { "Oppretter behandling ${behandling.id}" }
183+
sikkerlogg.info { "Oppretter behandling ${behandling.id}" }
186184

187185
val nå = nå()
188186

‎app/src/main/kotlin/no/nav/tiltakspenger/vedtak/repository/sak/PersonopplysningerBarnMedIdentDAO.kt

+2-4
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ package no.nav.tiltakspenger.vedtak.repository.sak
33
import kotliquery.Row
44
import kotliquery.Session
55
import kotliquery.queryOf
6-
import mu.KotlinLogging
6+
import no.nav.tiltakspenger.felles.sikkerlogg
77
import no.nav.tiltakspenger.libs.common.Fnr
88
import no.nav.tiltakspenger.libs.common.SakId
99
import no.nav.tiltakspenger.libs.common.UlidBase.Companion.random
@@ -15,8 +15,6 @@ internal object PersonopplysningerBarnMedIdentDAO {
1515

1616
private const val ULID_PREFIX_BARN_MED_IDENT = "barnm"
1717

18-
private val securelog = KotlinLogging.logger("tjenestekall")
19-
2018
internal fun hent(
2119
sakId: SakId,
2220
session: Session,
@@ -27,7 +25,7 @@ internal object PersonopplysningerBarnMedIdentDAO {
2725
personopplysninger: PersonopplysningerBarnMedIdent,
2826
session: Session,
2927
) {
30-
securelog.info { "Lagre personopplysninger for barn med ident $personopplysninger" }
28+
sikkerlogg.info { "Lagre personopplysninger for barn med ident $personopplysninger" }
3129
session.run(
3230
queryOf(
3331
lagreSql,

‎app/src/main/kotlin/no/nav/tiltakspenger/vedtak/repository/sak/PersonopplysningerBarnUtenIdentDAO.kt

+2-4
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ package no.nav.tiltakspenger.vedtak.repository.sak
33
import kotliquery.Row
44
import kotliquery.Session
55
import kotliquery.queryOf
6-
import mu.KotlinLogging
6+
import no.nav.tiltakspenger.felles.sikkerlogg
77
import no.nav.tiltakspenger.libs.common.SakId
88
import no.nav.tiltakspenger.libs.common.UlidBase.Companion.random
99
import no.nav.tiltakspenger.saksbehandling.domene.personopplysninger.PersonopplysningerBarnUtenIdent
@@ -13,8 +13,6 @@ internal object PersonopplysningerBarnUtenIdentDAO {
1313

1414
private const val ULID_PREFIX_BARN_UTEN_IDENT = "barnu"
1515

16-
private val securelog = KotlinLogging.logger("tjenestekall")
17-
1816
internal fun hent(
1917
sakId: SakId,
2018
session: Session,
@@ -25,7 +23,7 @@ internal object PersonopplysningerBarnUtenIdentDAO {
2523
personopplysninger: PersonopplysningerBarnUtenIdent,
2624
session: Session,
2725
) {
28-
securelog.info { "Lagre personopplysninger for barn uten ident $personopplysninger" }
26+
sikkerlogg.info { "Lagre personopplysninger for barn uten ident $personopplysninger" }
2927
session.run(
3028
queryOf(
3129
lagreSql,

‎app/src/main/kotlin/no/nav/tiltakspenger/vedtak/repository/sak/PersonopplysningerPostgresRepo.kt

+2-2
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@ import kotliquery.Session
55
import kotliquery.TransactionalSession
66
import kotliquery.queryOf
77
import mu.KotlinLogging
8+
import no.nav.tiltakspenger.felles.sikkerlogg
89
import no.nav.tiltakspenger.libs.common.Fnr
910
import no.nav.tiltakspenger.libs.common.SakId
1011
import no.nav.tiltakspenger.libs.common.UlidBase
@@ -25,7 +26,6 @@ class PersonopplysningerPostgresRepo(
2526

2627
companion object {
2728
private val log = KotlinLogging.logger {}
28-
private val securelog = KotlinLogging.logger("tjenestekall")
2929

3030
fun hentForSakId(
3131
sakId: SakId,
@@ -73,7 +73,7 @@ class PersonopplysningerPostgresRepo(
7373
personopplysninger: PersonopplysningerSøker,
7474
session: Session,
7575
) {
76-
securelog.info { "Lagre personopplysninger for søker $personopplysninger" }
76+
sikkerlogg.info { "Lagre personopplysninger for søker $personopplysninger" }
7777
session.run(
7878
queryOf(
7979
lagreSql,

‎app/src/main/kotlin/no/nav/tiltakspenger/vedtak/repository/sak/SakPostgresRepo.kt

+5-5
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,8 @@ package no.nav.tiltakspenger.vedtak.repository.sak
33
import kotliquery.Row
44
import kotliquery.Session
55
import kotliquery.queryOf
6-
import mu.KotlinLogging
76
import no.nav.tiltakspenger.felles.nå
7+
import no.nav.tiltakspenger.felles.sikkerlogg
88
import no.nav.tiltakspenger.libs.common.BehandlingId
99
import no.nav.tiltakspenger.libs.common.Fnr
1010
import no.nav.tiltakspenger.libs.common.SakId
@@ -192,7 +192,6 @@ internal class SakPostgresRepo(
192192
}
193193

194194
companion object {
195-
private val SECURELOG = KotlinLogging.logger("tjenestekall")
196195

197196
private fun hentSistEndret(
198197
sakId: SakId,
@@ -210,7 +209,7 @@ internal class SakPostgresRepo(
210209
sak: Sak,
211210
session: Session,
212211
): Sak {
213-
SECURELOG.info { "Oppdaterer sak ${sak.id}" }
212+
sikkerlogg.info { "Oppdaterer sak ${sak.id}" }
214213

215214
val antRaderOppdatert =
216215
session.run(
@@ -234,7 +233,7 @@ internal class SakPostgresRepo(
234233
sak: Sak,
235234
session: Session,
236235
): Sak {
237-
SECURELOG.info { "Oppretter sak ${sak.id}" }
236+
sikkerlogg.info { "Oppretter sak ${sak.id}" }
238237

239238
val nå = nå()
240239

@@ -262,7 +261,8 @@ internal class SakPostgresRepo(
262261
behandlinger = behandlinger,
263262
personopplysninger = PersonopplysningerPostgresRepo.hentForSakId(id, session),
264263
vedtak = RammevedtakPostgresRepo.hentForSakId(id, sessionContext),
265-
meldekort = MeldekortPostgresRepo.hentForSakId(id, session) ?: Meldeperioder.empty(behandlinger.first().tiltakstype),
264+
meldekort = MeldekortPostgresRepo.hentForSakId(id, session)
265+
?: Meldeperioder.empty(behandlinger.first().tiltakstype),
266266
)
267267
}
268268
}

‎app/src/main/kotlin/no/nav/tiltakspenger/vedtak/repository/vedtak/RammevedtakPostgresRepo.kt

+2-4
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ package no.nav.tiltakspenger.vedtak.repository.vedtak
33
import kotliquery.Row
44
import kotliquery.Session
55
import kotliquery.queryOf
6-
import mu.KotlinLogging
6+
import no.nav.tiltakspenger.felles.sikkerlogg
77
import no.nav.tiltakspenger.libs.common.BehandlingId
88
import no.nav.tiltakspenger.libs.common.Fnr
99
import no.nav.tiltakspenger.libs.common.SakId
@@ -20,8 +20,6 @@ import no.nav.tiltakspenger.saksbehandling.ports.RammevedtakRepo
2020
import no.nav.tiltakspenger.vedtak.repository.behandling.BehandlingPostgresRepo
2121
import java.time.LocalDateTime
2222

23-
private val SECURELOG = KotlinLogging.logger("tjenestekall")
24-
2523
class RammevedtakPostgresRepo(
2624
private val sessionFactory: PostgresSessionFactory,
2725
) : RammevedtakRepo {
@@ -60,7 +58,7 @@ class RammevedtakPostgresRepo(
6058
}.asList,
6159
)
6260
}.also {
63-
SECURELOG.info { "Hentet ${it.size} vedtak for ident $fnr" }
61+
sikkerlogg.info { "Hentet ${it.size} vedtak for ident $fnr" }
6462
}
6563

6664
override fun lagre(

‎app/src/main/kotlin/no/nav/tiltakspenger/vedtak/routes/VedtakApi.kt

+3-3
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,7 @@ import io.ktor.server.response.respondText
2525
import io.ktor.server.routing.routing
2626
import mu.KotlinLogging
2727
import no.nav.tiltakspenger.felles.exceptions.IkkeImplementertException
28+
import no.nav.tiltakspenger.felles.sikkerlogg
2829
import no.nav.tiltakspenger.libs.common.Rolle
2930
import no.nav.tiltakspenger.libs.common.Roller
3031
import no.nav.tiltakspenger.vedtak.AdRolle
@@ -44,7 +45,6 @@ import java.net.URI
4445
import java.util.UUID
4546

4647
private val LOG = KotlinLogging.logger {}
47-
private val SECURELOG = KotlinLogging.logger("tjenestekall")
4848

4949
internal fun Application.vedtakApi(
5050
config: Configuration.TokenVerificationConfig,
@@ -123,7 +123,7 @@ private fun AuthenticationConfig.jwt(
123123
realm: String,
124124
roles: Roller? = null,
125125
) = jwt(name) {
126-
SECURELOG.debug { "config : $config" }
126+
sikkerlogg.debug { "config : $config" }
127127
this.realm = realm
128128
val jwkProviderGammel = UrlJwkProvider(URI(config.jwksUri).toURL())
129129
verifier(jwkProviderGammel, config.issuer) {
@@ -168,7 +168,7 @@ private fun AuthenticationConfig.jwtSystemToken(
168168
realm: String,
169169
roles: Roller? = null,
170170
) = jwt(name) {
171-
SECURELOG.info { "config : $config" }
171+
sikkerlogg.info { "config : $config" }
172172
this.realm = realm
173173
val jwkProviderGammel = UrlJwkProvider(URI(config.jwksUri).toURL())
174174
verifier(jwkProviderGammel, config.issuer) {

0 commit comments

Comments
 (0)