Get rid of warning when creating tmp file for pdf.

navikt · Oct 1, 2024 · 365bab7 · 365bab7
1 parent 9817b60
commit 365bab7
Showing 1 changed file with 13 additions and 2 deletions.
diff --git a/src/main/kotlin/no/nav/klage/oppgave/service/DokumentService.kt b/src/main/kotlin/no/nav/klage/oppgave/service/DokumentService.kt
@@ -39,6 +39,7 @@ import org.springframework.http.MediaType
 import org.springframework.stereotype.Service
 import org.springframework.transaction.annotation.Transactional
 import reactor.core.publisher.Flux
+import java.io.File
 import java.io.IOException
 import java.math.BigInteger
 import java.nio.file.Files
@@ -202,7 +203,8 @@ class DokumentService(
 
     fun changeTitleInPDF(resource: Resource, title: String): Resource {
         try {
-            val tmpFile = Files.createTempFile(null, null).toFile()
+            val tmpFile = getFileToUse()
+
             val timeMillis = measureTimeMillis {
                 val memorySettingsForPDFBox: Long = 50_000_000
                 val document: PDDocument = if (resource is FileSystemResource) {
@@ -232,6 +234,16 @@ class DokumentService(
         }
     }
 
+    private fun getFileToUse(): File {
+        //TODO: is there a better way to do this?
+        val tmpFile = Files.createTempFile(null, null)
+        val pathToFile = tmpFile.toAbsolutePath().toString()
+
+        Files.delete(tmpFile)
+
+        return File(pathToFile)
+    }
+
     fun getDokumentReferanse(journalpostId: String, behandling: Behandling): DokumentReferanse {
         val journalpost = safFacade.getJournalposter(
             journalpostIdSet = setOf(journalpostId),
@@ -386,7 +398,6 @@ class DokumentService(
         merger.destinationDocumentInformation = pdDocumentInformation
 
         val pathToMergedDocument = Files.createTempFile(null, null)
-        pathToMergedDocument.toFile().deleteOnExit()
 
         merger.destinationFileName = pathToMergedDocument.toString()