From 847a1ca82a503b4c8cae297c2d84118199a91266 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?J=C3=B8rn-Are=20Flaten?= Date: Mon, 2 Dec 2024 15:59:46 +0100 Subject: [PATCH] chore: fixing excess param Co-authored-by: Andreas --- .../papirsykmelding/NasjonalSykmeldingService.kt | 2 +- .../api/NasjonalOppgaveController.kt | 11 +++++------ .../tilgangskontroll/OppgaveSecurityService.kt | 16 ++++------------ 3 files changed, 10 insertions(+), 19 deletions(-) diff --git a/src/main/kotlin/no/nav/sykdig/digitalisering/papirsykmelding/NasjonalSykmeldingService.kt b/src/main/kotlin/no/nav/sykdig/digitalisering/papirsykmelding/NasjonalSykmeldingService.kt index 446a1fe7..9c2ff9d3 100644 --- a/src/main/kotlin/no/nav/sykdig/digitalisering/papirsykmelding/NasjonalSykmeldingService.kt +++ b/src/main/kotlin/no/nav/sykdig/digitalisering/papirsykmelding/NasjonalSykmeldingService.kt @@ -114,7 +114,7 @@ class NasjonalSykmeldingService( ): ResponseEntity { if (validationResult.status == Status.OK || validationResult.status == Status.MANUAL_PROCESSING) { // val veileder = oppgaveSecurityService.getNavIdent() - val veileder = Veileder(oppgaveSecurityService.getNavEmailAsync()) + val veileder = Veileder(oppgaveSecurityService.getNavEmail()) if (ferdigstillRegistrering.oppgaveId != null) { journalpostService.ferdigstillNasjonalJournalpost( ferdigstillRegistrering = ferdigstillRegistrering, diff --git a/src/main/kotlin/no/nav/sykdig/digitalisering/papirsykmelding/api/NasjonalOppgaveController.kt b/src/main/kotlin/no/nav/sykdig/digitalisering/papirsykmelding/api/NasjonalOppgaveController.kt index 0c9bda8d..584ae2f4 100644 --- a/src/main/kotlin/no/nav/sykdig/digitalisering/papirsykmelding/api/NasjonalOppgaveController.kt +++ b/src/main/kotlin/no/nav/sykdig/digitalisering/papirsykmelding/api/NasjonalOppgaveController.kt @@ -11,7 +11,6 @@ import no.nav.sykdig.digitalisering.papirsykmelding.db.model.NasjonalManuellOppg import no.nav.sykdig.digitalisering.pdl.Navn import no.nav.sykdig.digitalisering.pdl.PersonService import no.nav.sykdig.securelog -import org.springframework.http.HttpStatus import org.springframework.http.HttpStatusCode import org.springframework.http.ResponseEntity import org.springframework.security.access.prepost.PreAuthorize @@ -50,16 +49,16 @@ class NasjonalOppgaveController( return nasjonalOppgaveService.avvisOppgave(oppgaveId.toInt(), avvisSykmeldingRequest, authorization, navEnhet) } - @GetMapping("/oppgave/{oppgaveid}") + @GetMapping("/oppgave/{oppgaveId}") @PreAuthorize("@oppgaveSecurityService.hasAccessToNasjonalOppgave(#oppgaveId)") @ResponseBody fun getPapirsykmeldingManuellOppgave( - @PathVariable oppgaveid: String, + @PathVariable oppgaveId: String, @RequestHeader("Authorization") authorization: String, ): ResponseEntity { log.info("Current thread: ${Thread.currentThread().name}") - log.info("papirsykmelding: henter oppgave med id $oppgaveid gjennom syk-dig proxy") - val oppgave = smregistreringClient.getOppgaveRequest(authorization, oppgaveid) + log.info("papirsykmelding: henter oppgave med id $oppgaveId gjennom syk-dig proxy") + val oppgave = smregistreringClient.getOppgaveRequest(authorization, oppgaveId) val papirManuellOppgave = oppgave.body if (papirManuellOppgave != null) { securelog.info("lagrer nasjonalOppgave i db $papirManuellOppgave") @@ -103,7 +102,7 @@ class NasjonalOppgaveController( @PreAuthorize("@oppgaveSecurityService.hasAccessToNasjonalOppgave(#oppgaveId)") @ResponseBody suspend fun sendOppgave( - @PathVariable oppgaveId: Int, + @PathVariable oppgaveId: String, @RequestHeader("Authorization") authorization: String, @RequestHeader("X-Nav-Enhet") navEnhet: String, @RequestBody papirSykmelding: SmRegistreringManuell, diff --git a/src/main/kotlin/no/nav/sykdig/digitalisering/tilgangskontroll/OppgaveSecurityService.kt b/src/main/kotlin/no/nav/sykdig/digitalisering/tilgangskontroll/OppgaveSecurityService.kt index e364d0a7..0d739268 100644 --- a/src/main/kotlin/no/nav/sykdig/digitalisering/tilgangskontroll/OppgaveSecurityService.kt +++ b/src/main/kotlin/no/nav/sykdig/digitalisering/tilgangskontroll/OppgaveSecurityService.kt @@ -43,18 +43,17 @@ class OppgaveSecurityService( return tilgang } - suspend fun hasAccessToNasjonalOppgave(oppgaveId: String): Boolean = withContext( - Dispatchers.IO) { + fun hasAccessToNasjonalOppgave(oppgaveId: String): Boolean { securelog.info("sjekker om bruker har tilgang på oppgave $oppgaveId") val oppgave = nasjonalOppgaveRepository.findByOppgaveId(oppgaveId.toInt()) - val navEmail = getNavEmailAsync() + val navEmail = getNavEmail() val fnr = oppgave?.fnr if (oppgave != null && fnr != null) { val tilgang = hasAccess(fnr, navEmail) securelog.info("Innlogget bruker: $navEmail har${if (!tilgang) " ikke" else ""} tilgang til oppgave med id $oppgaveId") - tilgang + return tilgang } - false + return false } fun hasAccessToSykmelding(sykmeldingId: String): Boolean { @@ -129,13 +128,6 @@ class OppgaveSecurityService( val authentication = SecurityContextHolder.getContext().authentication as JwtAuthenticationToken return authentication.token.claims["preferred_username"].toString() } - - suspend fun getNavEmailAsync(): String { - return ReactiveSecurityContextHolder.getContext().map { context -> - val auth = context.authentication as JwtAuthenticationToken - auth.token.claims["preferred_username"].toString() - }.awaitSingle() - } }