marcus/pdf-umbenenner
1
0
Fork 0
Code Issues 14 Releases 2 Activity

Fix #58: Verschluckte Exception in ResolveHistoricalFileName loggen

Browse Source 
Catch von Exception auf RuntimeException eingeengt; unerwartete
Laufzeitfehler werden jetzt per logger.warn() protokolliert und
weiterpropagiert statt still verschluckt zu werden.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
Marcus van Elst
2026-04-28 15:43:52 +02:00
parent 43c54923f8
commit 4bbee57d41
1 changed files with 20 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files
pdf-umbenenner-application/src/main/java/de/gecheckt/pdf/umbenenner/application/usecase/DefaultResolveHistoricalFileNameUseCase.java
+20 -11
View File
@@ -3,6 +3,9 @@ package de.gecheckt.pdf.umbenenner.application.usecase;
import java.util.Objects; import java.util.Objects;
import java.util.Optional; import java.util.Optional;
import org.apache.logging.log4j.LogManager;
import org.apache.logging.log4j.Logger;
import de.gecheckt.pdf.umbenenner.application.port.in.ResolveHistoricalFileNameUseCase; import de.gecheckt.pdf.umbenenner.application.port.in.ResolveHistoricalFileNameUseCase;
import de.gecheckt.pdf.umbenenner.application.port.out.DocumentRecordLookupResult; import de.gecheckt.pdf.umbenenner.application.port.out.DocumentRecordLookupResult;
import de.gecheckt.pdf.umbenenner.application.port.out.DocumentRecordRepository; import de.gecheckt.pdf.umbenenner.application.port.out.DocumentRecordRepository;
@@ -10,20 +13,24 @@ import de.gecheckt.pdf.umbenenner.application.port.out.DocumentTerminalSuccess;
import de.gecheckt.pdf.umbenenner.domain.model.DocumentFingerprint; import de.gecheckt.pdf.umbenenner.domain.model.DocumentFingerprint;
/** /**
* Default implementation of {@link ResolveHistoricalFileNameUseCase}. * Standardimplementierung von {@link ResolveHistoricalFileNameUseCase}.
* <p> * <p>
* Queries the {@link DocumentRecordRepository} for the master record of the given fingerprint. * Fragt den {@link DocumentRecordRepository} nach dem Stammsatz des angegebenen
* If the record represents a document that previously reached a successful terminal state, * Fingerprints ab. Ist der Stammsatz terminal erfolgreich, wird der zuletzt
* the last known target filename ({@code lastTargetFileName}) is returned. * geschriebene Zieldateiname zurückgegeben.
* <p> * <p>
* For all other terminal states (e.g. documents that finally failed without ever producing * Für alle anderen terminalen Zustände oder wenn kein Stammsatz vorhanden ist,
* a target copy) or when no master record exists, an empty {@link Optional} is returned. * wird ein leeres {@link Optional} zurückgegeben.
* Technical failures during the repository lookup are caught silently and treated as * Unerwartete {@link RuntimeException}s aus dem Repository werden geloggt (WARN) und
* an absent result so that the calling GUI layer is never forced to handle exceptions * weiterpropagiert. Erwartete Lookup-Fehler werden als
* from this query path. * {@code PersistenceLookupTechnicalFailure} im Rückgabewert kodiert und führen
* zu einem leeren {@link Optional}.
*/ */
public class DefaultResolveHistoricalFileNameUseCase implements ResolveHistoricalFileNameUseCase { public class DefaultResolveHistoricalFileNameUseCase implements ResolveHistoricalFileNameUseCase {
private static final Logger logger =
LogManager.getLogger(DefaultResolveHistoricalFileNameUseCase.class);
private final DocumentRecordRepository documentRecordRepository; private final DocumentRecordRepository documentRecordRepository;
/** /**
@@ -62,8 +69,10 @@ public class DefaultResolveHistoricalFileNameUseCase implements ResolveHistorica
return Optional.ofNullable(success.record().lastTargetFileName()); return Optional.ofNullable(success.record().lastTargetFileName());
} }
return Optional.empty(); return Optional.empty();
} catch (Exception e) { } catch (RuntimeException e) {
return Optional.empty(); logger.warn("Unerwarteter Fehler beim Lesen des historischen Dateinamens für Fingerprint {}: {}",
fingerprint, e.getMessage(), e);
throw e;
} }
} }
} }