From b242fd66d0033286f0185e04be2b905e3770c247 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?J=C3=B8rn-Are=20Flaten?= <ja.flaten91@gmail.com>
Date: Thu, 5 Dec 2024 12:48:53 +0100
Subject: [PATCH] chore: checking for invalid oppgaveId in getOppgaveRequest

---
 .../papirsykmelding/NasjonalOppgaveService.kt            | 9 +++++----
 .../papirsykmelding/api/SmregistreringClient.kt          | 3 +++
 2 files changed, 8 insertions(+), 4 deletions(-)

diff --git a/src/main/kotlin/no/nav/sykdig/digitalisering/papirsykmelding/NasjonalOppgaveService.kt b/src/main/kotlin/no/nav/sykdig/digitalisering/papirsykmelding/NasjonalOppgaveService.kt
index c1303f18..eef6f1a8 100644
--- a/src/main/kotlin/no/nav/sykdig/digitalisering/papirsykmelding/NasjonalOppgaveService.kt
+++ b/src/main/kotlin/no/nav/sykdig/digitalisering/papirsykmelding/NasjonalOppgaveService.kt
@@ -67,10 +67,6 @@ class NasjonalOppgaveService(
         return updated
     }
 
-    fun isValidOppgaveId(oppgaveId: String): Boolean {
-        val regex = Regex("^\\d{9}$|^[a-zA-Z0-9]{1,20}$")
-        return oppgaveId.matches(regex)
-    }
 
     fun findByOppgaveId(oppgaveId: String): NasjonalManuellOppgaveDAO? {
         if(!isValidOppgaveId(oppgaveId))
@@ -269,3 +265,8 @@ private fun getLoggingMeta(sykmeldingId: String, oppgave: NasjonalManuellOppgave
     )
 }
 }
+
+fun isValidOppgaveId(oppgaveId: String): Boolean {
+    val regex = Regex("^\\d{9}$|^[a-zA-Z0-9]{1,20}$")
+    return oppgaveId.matches(regex)
+}
\ No newline at end of file
diff --git a/src/main/kotlin/no/nav/sykdig/digitalisering/papirsykmelding/api/SmregistreringClient.kt b/src/main/kotlin/no/nav/sykdig/digitalisering/papirsykmelding/api/SmregistreringClient.kt
index 3b401e1e..cf33605f 100644
--- a/src/main/kotlin/no/nav/sykdig/digitalisering/papirsykmelding/api/SmregistreringClient.kt
+++ b/src/main/kotlin/no/nav/sykdig/digitalisering/papirsykmelding/api/SmregistreringClient.kt
@@ -5,6 +5,7 @@ import no.nav.sykdig.digitalisering.papirsykmelding.api.model.AvvisSykmeldingReq
 import no.nav.sykdig.digitalisering.papirsykmelding.api.model.PapirManuellOppgave
 import no.nav.sykdig.digitalisering.papirsykmelding.api.model.SmRegistreringManuell
 import no.nav.sykdig.digitalisering.papirsykmelding.api.model.Sykmelder
+import no.nav.sykdig.digitalisering.papirsykmelding.isValidOppgaveId
 import org.springframework.beans.factory.annotation.Value
 import org.springframework.http.HttpEntity
 import org.springframework.http.HttpHeaders
@@ -57,6 +58,8 @@ class SmregistreringClient(
         authorization: String,
         oppgaveId: String,
     ): ResponseEntity<PapirManuellOppgave> {
+        if(!isValidOppgaveId(oppgaveId))
+            throw IllegalArgumentException("Invalid oppgaveId does not contain only alphanumerical characters. oppgaveId: $oppgaveId")
         val headers = HttpHeaders()
         headers.contentType = MediaType.APPLICATION_JSON
         headers.setBearerAuth(removeBearerPrefix(authorization))