Changed behaviour

apps/processer/src/main/kotlin/no/iktdev/mediaprocessing/processer/Reporter.kt

@@ -1,5 +1,6 @@
 package no.iktdev.mediaprocessing.processer
 
+import mu.KotlinLogging
 import no.iktdev.mediaprocessing.shared.common.SharedConfig
 import no.iktdev.mediaprocessing.shared.contract.dto.ProcesserEventInfo
 import org.springframework.beans.factory.annotation.Autowired
@@ -13,12 +14,15 @@ class Reporter() {
     lateinit var restTemplate: RestTemplate
     @Autowired
     lateinit var messageTemplate: SimpMessagingTemplate
+
+    private val log = KotlinLogging.logger {}
+
     fun sendEncodeProgress(progress: ProcesserEventInfo) {
         try {
             restTemplate.postForEntity(SharedConfig.uiUrl + "/encode/progress", progress, String::class.java)
             messageTemplate.convertAndSend("/topic/encode/progress", progress)
         } catch (e: Exception) {
-            e.printStackTrace()
+            log.error { e.message }
         }
     }
 
@@ -27,7 +31,7 @@ class Reporter() {
             restTemplate.postForEntity(SharedConfig.uiUrl + "/extract/progress", progress, String::class.java)
             messageTemplate.convertAndSend("/topic/extract/progress", progress)
         } catch (e: Exception) {
-            e.printStackTrace()
+            log.error { e.message }
         }
     }
 

apps/processer/src/main/kotlin/no/iktdev/mediaprocessing/processer/services/EncodeService.kt

@@ -83,7 +83,8 @@ class EncodeService(@Autowired override var coordinator: Coordinator, @Autowired
 
     fun startEncode(event: PersistentProcessDataMessage) {
         val ffwrc = event.data as FfmpegWorkRequestCreated
-        File(ffwrc.outFile).parentFile.mkdirs()
+        val outFile = File(ffwrc.outFile)
+        outFile.parentFile.mkdirs()
         if (!logDir.exists()) {
             logDir.mkdirs()
         }
@@ -92,12 +93,14 @@ class EncodeService(@Autowired override var coordinator: Coordinator, @Autowired
         if (setClaim) {
             log.info { "Claim successful for ${event.referenceId} encode" }
             runner = FfmpegWorker(event.referenceId, event.eventId, info = ffwrc, logDir = logDir, listener = ffmpegWorkerEvents )
-            if (File(ffwrc.outFile).exists() && ffwrc.arguments.firstOrNull() != "-y") {
+            if (outFile.exists()) {
+                if (ffwrc.arguments.firstOrNull() != "-y") {
                     ffmpegWorkerEvents.onError(event.referenceId, event.eventId, ffwrc, "${this::class.java.simpleName} identified the file as already existing, either allow overwrite or delete the offending file: ${ffwrc.outFile}")
                     // Setting consumed to prevent spamming
-                eventManager.setProcessEventCompleted(event.referenceId, event.eventId)
+                    eventManager.setProcessEventCompleted(event.referenceId, event.eventId, Status.ERROR)
                     return
                 }
+            }
             runner?.runWithProgress()
 
         } else {
@@ -196,7 +199,11 @@ class EncodeService(@Autowired override var coordinator: Coordinator, @Autowired
             outputFiles = listOf(info.outFile),
             progress = progress?.toProcessProgress()
         )
+        try {
             reporter.sendEncodeProgress(processerEventInfo)
+        } catch (e: Exception) {
+            e.printStackTrace()
+        }
     }
 
 

shared/common/src/main/kotlin/no/iktdev/mediaprocessing/shared/common/persistance/PersistentEventManager.kt

@@ -142,7 +142,8 @@ class PersistentEventManager(private val dataSource: DataSource) {
     }
 
     fun isProcessEventClaimed(referenceId: String, eventId: String): Boolean {
-        return getProcessEventWith(referenceId, eventId)?.claimed ?: false
+        val info = getProcessEventWith(referenceId, eventId)
+        return info?.claimed ?: true && info?.consumed ?: true
     }
 
     fun isProcessEventCompleted(referenceId: String, eventId: String): Boolean {