chore: checking for invalid oppgaveId in getOppgaveRequest

navikt · Dec 5, 2024 · b242fd6 · b242fd6
1 parent 79e8ff6
commit b242fd6
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 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
@@ -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)
+}
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
@@ -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))