From 2ac1234f3d984507ae9cb4e8eac88b9dbf1b45e3 Mon Sep 17 00:00:00 2001 From: Qadeer Khan Date: Mon, 17 Mar 2025 19:26:56 +0100 Subject: [PATCH 1/2] Potential fix for code scanning alert no. 2: Use of externally-controlled format string Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com> --- .../main/java/no/nav/ung/sak/kontrakt/dokument/DokumentDto.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kontrakt/src/main/java/no/nav/ung/sak/kontrakt/dokument/DokumentDto.java b/kontrakt/src/main/java/no/nav/ung/sak/kontrakt/dokument/DokumentDto.java index 3e1e2c5686..d9be545e38 100644 --- a/kontrakt/src/main/java/no/nav/ung/sak/kontrakt/dokument/DokumentDto.java +++ b/kontrakt/src/main/java/no/nav/ung/sak/kontrakt/dokument/DokumentDto.java @@ -122,7 +122,7 @@ public int hashCode() { void genererLenke() { if (journalpostId != null && journalpostId.getVerdi() != null && dokumentId != null) { - this.href = String.format(basePath, journalpostId.getVerdi(), dokumentId); + this.href = String.format("%s", basePath) + String.format("%s/%s", journalpostId.getVerdi(), dokumentId); } } } From 8996908e5e4718e4d1d34f4dbc7ac98d7bfa8100 Mon Sep 17 00:00:00 2001 From: Qadeer Khan Date: Mon, 17 Mar 2025 19:36:41 +0100 Subject: [PATCH 2/2] fix som faktisk funker. Godt med tester :) --- .../java/no/nav/ung/sak/kontrakt/dokument/DokumentDto.java | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/kontrakt/src/main/java/no/nav/ung/sak/kontrakt/dokument/DokumentDto.java b/kontrakt/src/main/java/no/nav/ung/sak/kontrakt/dokument/DokumentDto.java index d9be545e38..0886b2b336 100644 --- a/kontrakt/src/main/java/no/nav/ung/sak/kontrakt/dokument/DokumentDto.java +++ b/kontrakt/src/main/java/no/nav/ung/sak/kontrakt/dokument/DokumentDto.java @@ -12,7 +12,7 @@ import no.nav.ung.sak.typer.JournalpostId; public class DokumentDto { - private List behandlinger = new ArrayList<>(); + private List behandlinger; private String dokumentId; private String gjelderFor; private JournalpostId journalpostId; @@ -25,7 +25,7 @@ public class DokumentDto { private String brevkode; public DokumentDto(String basePath) { - this.basePath = basePath + "&journalpostId=%s&dokumentId=%s"; + this.basePath = String.format("%s",basePath) + "&journalpostId=%s&dokumentId=%s"; this.behandlinger = new ArrayList<>(); } @@ -122,7 +122,8 @@ public int hashCode() { void genererLenke() { if (journalpostId != null && journalpostId.getVerdi() != null && dokumentId != null) { - this.href = String.format("%s", basePath) + String.format("%s/%s", journalpostId.getVerdi(), dokumentId); + this.href = String.format(basePath, journalpostId.getVerdi(), dokumentId); } + } }