From 7f2e4bd0876dc715e21017619af9155ca96477a9 Mon Sep 17 00:00:00 2001 From: Thomas Johansen Date: Thu, 19 Sep 2024 07:27:02 +0200 Subject: [PATCH] Justerte mock data, log debug for sikkerhet --- .../api/services/BekreftelseService.kt | 7 +- .../paw/bekreftelse/api/services/MockData.kt | 87 ------------------- .../api/services/MockDataService.kt | 62 +++++++++++++ .../resources/local/application_config.toml | 1 - .../src/main/resources/logback.xml | 17 +--- .../resources/nais/application_config.toml | 1 - .../src/main/resources/test/mock-data.toml | 38 ++++++++ 7 files changed, 104 insertions(+), 109 deletions(-) delete mode 100644 apps/bekreftelse-api/src/main/kotlin/no/nav/paw/bekreftelse/api/services/MockData.kt create mode 100644 apps/bekreftelse-api/src/main/kotlin/no/nav/paw/bekreftelse/api/services/MockDataService.kt create mode 100644 apps/bekreftelse-api/src/main/resources/test/mock-data.toml diff --git a/apps/bekreftelse-api/src/main/kotlin/no/nav/paw/bekreftelse/api/services/BekreftelseService.kt b/apps/bekreftelse-api/src/main/kotlin/no/nav/paw/bekreftelse/api/services/BekreftelseService.kt index ba3b4321..8e4d1ca4 100644 --- a/apps/bekreftelse-api/src/main/kotlin/no/nav/paw/bekreftelse/api/services/BekreftelseService.kt +++ b/apps/bekreftelse-api/src/main/kotlin/no/nav/paw/bekreftelse/api/services/BekreftelseService.kt @@ -25,14 +25,13 @@ import org.apache.kafka.streams.StoreQueryParameters import org.apache.kafka.streams.state.QueryableStoreTypes import org.apache.kafka.streams.state.ReadOnlyKeyValueStore -private val mockData = MockData() - class BekreftelseService( private val applicationConfig: ApplicationConfig, private val httpClient: HttpClient, private val kafkaStreams: KafkaStreams, private val bekreftelseProducer: BekreftelseProducer ) { + private val mockDataService = MockDataService() private var internStateStore: ReadOnlyKeyValueStore? = null private fun getInternStateStore(): ReadOnlyKeyValueStore { @@ -59,7 +58,7 @@ class BekreftelseService( ): TilgjengeligBekreftelserResponse { // TODO Fjern når vi har ferdig Kafka-logikk if (useMockData) { - return mockData.finnTilgjengeligBekreftelser(sluttbruker.identitetsnummer) + return mockDataService.finnTilgjengeligBekreftelser(sluttbruker.identitetsnummer) } val internState = getInternStateStore().get(sluttbruker.arbeidssoekerId) @@ -81,7 +80,7 @@ class BekreftelseService( ) { // TODO Fjern når vi har ferdig Kafka-logikk if (useMockData) { - return mockData.mottaBekreftelse(sluttbruker.identitetsnummer, request.bekreftelseId) + return mockDataService.mottaBekreftelse(sluttbruker.identitetsnummer, request.bekreftelseId) } val internState = getInternStateStore().get(sluttbruker.arbeidssoekerId) diff --git a/apps/bekreftelse-api/src/main/kotlin/no/nav/paw/bekreftelse/api/services/MockData.kt b/apps/bekreftelse-api/src/main/kotlin/no/nav/paw/bekreftelse/api/services/MockData.kt deleted file mode 100644 index 59122826..00000000 --- a/apps/bekreftelse-api/src/main/kotlin/no/nav/paw/bekreftelse/api/services/MockData.kt +++ /dev/null @@ -1,87 +0,0 @@ -package no.nav.paw.bekreftelse.api.services - -import no.nav.paw.bekreftelse.api.model.TilgjengeligBekreftelse -import no.nav.paw.bekreftelse.api.model.TilgjengeligBekreftelserResponse -import java.time.Duration -import java.time.Instant -import java.util.* - -// TODO Fjern når vi har ferdig Kafka-logikk -class MockData { - - private var tilgjengeligBekreftelser = mutableMapOf() - private val fnr1 = "17830348441" - private val fnr2 = "19519238708" - private val fnr3 = "02837797848" - private val fnr4 = "16868598968" - private val fnr5 = "28878098821" - private val periodeId1 = UUID.fromString("84201f96-363b-4aab-a589-89fa4b9b1feb") - private val periodeId2 = UUID.fromString("ec6b5a10-b67c-42c1-b6e7-a642c36bd78e") - private val periodeId3 = UUID.fromString("44a4375c-b7ab-40ea-83f5-0eb9869925eb") - private val periodeId4: UUID = UUID.fromString("bbf3e9eb-6d7b-465b-bf79-ae6c82cf1ddd") - private val periodeId5: UUID = UUID.fromString("6ea57aec-353c-4df5-935f-9bead8afb221") - private val bekreftelseId1: UUID = UUID.fromString("f45ffbf3-e4d5-49fd-b5b7-17aaee478dfc") - private val bekreftelseId2: UUID = UUID.fromString("0cae8890-5500-4f5f-8fc1-9a0aae3b35a0") - private val bekreftelseId3: UUID = UUID.fromString("4f5e7f5c-1fe3-4b27-a07b-34ff9f4ea23f") - private val bekreftelseId4a: UUID = UUID.fromString("47e5c02d-abab-4e75-951c-db6c985901e4") - private val bekreftelseId4b: UUID = UUID.fromString("77322685-80db-41db-b79f-86915a9a5d9a") - private val bekreftelseId5a: UUID = UUID.fromString("992d5363-bab4-4b1d-987e-3e8eb4db3f64") - private val bekreftelseId5b: UUID = UUID.fromString("9777408c-938d-41e6-b9fd-5177120695d6") - - init { - tilgjengeligBekreftelser[fnr1] = listOf( - TilgjengeligBekreftelse(periodeId1, bekreftelseId1, pastInstant(), futureInstant()) - ) - tilgjengeligBekreftelser[fnr2] = listOf( - TilgjengeligBekreftelse(periodeId2, bekreftelseId2, pastInstant(), futureInstant()) - ) - tilgjengeligBekreftelser[fnr3] = listOf( - TilgjengeligBekreftelse(periodeId3, bekreftelseId3, pastInstant(), futureInstant()) - ) - val fra4a = pastInstant() - val fra4b = fra4a.minus(pastDuration()) - tilgjengeligBekreftelser[fnr4] = listOf( - TilgjengeligBekreftelse(periodeId4, bekreftelseId4a, fra4a, futureInstant()), - TilgjengeligBekreftelse(periodeId4, bekreftelseId4b, pastInstant(), fra4b) - ) - val fra5a = pastInstant() - val fra5b = fra5a.minus(pastDuration()) - tilgjengeligBekreftelser[fnr5] = listOf( - TilgjengeligBekreftelse(periodeId5, bekreftelseId5a, fra5a, futureInstant()), - TilgjengeligBekreftelse(periodeId5, bekreftelseId5b, fra5b, fra5a) - ) - } - - private fun pastInstant(): Instant { - return Instant.now().minus(pastDuration()) - } - - private fun futureInstant(): Instant { - return Instant.now().minus(futureDuration()) - } - - private fun pastDuration(): Duration { - val days = Random().nextLong(10, 90) - return Duration.ofDays(days) - } - - private fun futureDuration(): Duration { - val days = Random().nextLong(0, 90) - return Duration.ofDays(days) - } - - fun finnTilgjengeligBekreftelser(identitetsnummer: String): TilgjengeligBekreftelserResponse { - return tilgjengeligBekreftelser[identitetsnummer] ?: emptyList() - } - - fun mottaBekreftelse( - identitetsnummer: String, - bekreftelseId: UUID - ) { - val eksisterende = tilgjengeligBekreftelser[identitetsnummer] - if (eksisterende != null) { - val oppdatert = eksisterende.filter { it.bekreftelseId != bekreftelseId } - tilgjengeligBekreftelser[identitetsnummer] = oppdatert - } - } -} \ No newline at end of file diff --git a/apps/bekreftelse-api/src/main/kotlin/no/nav/paw/bekreftelse/api/services/MockDataService.kt b/apps/bekreftelse-api/src/main/kotlin/no/nav/paw/bekreftelse/api/services/MockDataService.kt new file mode 100644 index 00000000..585306f4 --- /dev/null +++ b/apps/bekreftelse-api/src/main/kotlin/no/nav/paw/bekreftelse/api/services/MockDataService.kt @@ -0,0 +1,62 @@ +package no.nav.paw.bekreftelse.api.services + +import no.nav.paw.bekreftelse.api.model.TilgjengeligBekreftelse +import no.nav.paw.bekreftelse.api.model.TilgjengeligBekreftelserResponse +import no.nav.paw.config.hoplite.loadConfigFromProvidedResource +import java.time.Duration +import java.time.Instant +import java.util.* + +data class MockBekreftelse( + val id: UUID +) + +data class MockPeriode( + val id: UUID, + val bekreftelser: List +) + +data class MockPerson( + val ident: String, + val perioder: List +) + +data class MockData( + val personer: List +) + +// TODO Fjern når vi har ferdig Kafka-logikk +class MockDataService { + + private var mockData = loadConfigFromProvidedResource("/test/mock-data.toml") + private var tilgjengeligBekreftelser = mutableMapOf() + + init { + mockData.personer.forEach { person -> + person.perioder.forEach { periode -> + val bekreftelser = periode.bekreftelser.map { bekreftelse -> + val days = Random().nextLong(1, 13) + val gjelderFra = Instant.now().minus(Duration.ofDays(days)) + val gjelserTil = gjelderFra.plus(Duration.ofDays(14)) + TilgjengeligBekreftelse(periode.id, bekreftelse.id, gjelderFra, gjelserTil) + } + tilgjengeligBekreftelser[person.ident] = bekreftelser + } + } + } + + fun finnTilgjengeligBekreftelser(identitetsnummer: String): TilgjengeligBekreftelserResponse { + return tilgjengeligBekreftelser[identitetsnummer] ?: emptyList() + } + + fun mottaBekreftelse( + identitetsnummer: String, + bekreftelseId: UUID + ) { + val eksisterende = tilgjengeligBekreftelser[identitetsnummer] + if (eksisterende != null) { + val oppdatert = eksisterende.filter { it.bekreftelseId != bekreftelseId } + tilgjengeligBekreftelser[identitetsnummer] = oppdatert + } + } +} \ No newline at end of file diff --git a/apps/bekreftelse-api/src/main/resources/local/application_config.toml b/apps/bekreftelse-api/src/main/resources/local/application_config.toml index ad552e97..0041d4ec 100644 --- a/apps/bekreftelse-api/src/main/resources/local/application_config.toml +++ b/apps/bekreftelse-api/src/main/resources/local/application_config.toml @@ -26,7 +26,6 @@ clientId = "paw-arbeidssoeker-bekreftelse-api" [authProviders.claims] map = ["NAVident"] -combineWithOr = false [azureM2M] tokenEndpointUrl = "http://localhost:8081/default/token" diff --git a/apps/bekreftelse-api/src/main/resources/logback.xml b/apps/bekreftelse-api/src/main/resources/logback.xml index 13ca21f2..06bb30df 100644 --- a/apps/bekreftelse-api/src/main/resources/logback.xml +++ b/apps/bekreftelse-api/src/main/resources/logback.xml @@ -10,19 +10,6 @@ - - /secure-logs/secure.log - - /secure-logs/secure.log.%i - 1 - 1 - - - 50MB - - - - @@ -38,12 +25,10 @@ - - - + diff --git a/apps/bekreftelse-api/src/main/resources/nais/application_config.toml b/apps/bekreftelse-api/src/main/resources/nais/application_config.toml index 26e2ca5e..458a0e70 100644 --- a/apps/bekreftelse-api/src/main/resources/nais/application_config.toml +++ b/apps/bekreftelse-api/src/main/resources/nais/application_config.toml @@ -26,7 +26,6 @@ clientId = "${AZURE_APP_CLIENT_ID}" [authProviders.claims] map = ["NAVident"] -combineWithOr = true [azureM2M] tokenEndpointUrl = "${AZURE_OPENID_CONFIG_TOKEN_ENDPOINT}" diff --git a/apps/bekreftelse-api/src/main/resources/test/mock-data.toml b/apps/bekreftelse-api/src/main/resources/test/mock-data.toml new file mode 100644 index 00000000..6fba5d9f --- /dev/null +++ b/apps/bekreftelse-api/src/main/resources/test/mock-data.toml @@ -0,0 +1,38 @@ +[[personer]] +ident = "17830348441" +[[personer.perioder]] +id = "84201f96-363b-4aab-a589-89fa4b9b1feb" +[[personer.perioder.bekreftelser]] +id = "f45ffbf3-e4d5-49fd-b5b7-17aaee478dfc" + +[[personer]] +ident = "19519238708" +[[personer.perioder]] +id = "ec6b5a10-b67c-42c1-b6e7-a642c36bd78e" +[[personer.perioder.bekreftelser]] +id = "0cae8890-5500-4f5f-8fc1-9a0aae3b35a0" + +[[personer]] +ident = "02837797848" +[[personer.perioder]] +id = "44a4375c-b7ab-40ea-83f5-0eb9869925eb" +[[personer.perioder.bekreftelser]] +id = "4f5e7f5c-1fe3-4b27-a07b-34ff9f4ea23f" + +[[personer]] +ident = "16868598968" +[[personer.perioder]] +id = "bbf3e9eb-6d7b-465b-bf79-ae6c82cf1ddd" +[[personer.perioder.bekreftelser]] +id = "47e5c02d-abab-4e75-951c-db6c985901e4" +[[personer.perioder.bekreftelser]] +id = "77322685-80db-41db-b79f-86915a9a5d9a" + +[[personer]] +ident = "28878098821" +[[personer.perioder]] +id = "6ea57aec-353c-4df5-935f-9bead8afb221" +[[personer.perioder.bekreftelser]] +id = "992d5363-bab4-4b1d-987e-3e8eb4db3f64" +[[personer.perioder.bekreftelser]] +id = "9777408c-938d-41e6-b9fd-5177120695d6"