Refactoring - Version stable

src/main/java/dev/lions/unionflow/server/exception/GlobalExceptionMapper.java

@@ -2,6 +2,9 @@ package dev.lions.unionflow.server.exception;
 
 import dev.lions.unionflow.server.service.SystemLoggingService;
 import jakarta.inject.Inject;
+import jakarta.ws.rs.ForbiddenException;
+import jakarta.ws.rs.NotAuthorizedException;
+import jakarta.ws.rs.NotFoundException;
 import jakarta.ws.rs.WebApplicationException;
 import jakarta.ws.rs.core.Context;
 import jakarta.ws.rs.core.Response;
@@ -33,12 +36,19 @@ public class GlobalExceptionMapper implements ExceptionMapper<Throwable> {
 
     @Override
     public Response toResponse(Throwable exception) {
-        // Logger l'exception dans les logs applicatifs
-        log.error("Unhandled exception", exception);
-
         // Déterminer le code HTTP
         int statusCode = determineStatusCode(exception);
 
+        // Les exceptions métier 4xx (404, 401, 403) sont des cas normaux :
+        // on les logue en DEBUG/WARN, sans stack trace et sans persister en system_logs.
+        if (isExpectedClientError(exception)) {
+            log.debug("Expected client error [{}]: {}", statusCode, exception.getMessage());
+            return buildErrorResponse(exception, statusCode);
+        }
+
+        // Pour toute autre exception (5xx ou inattendue) : log ERROR + persistance
+        log.error("Unhandled exception", exception);
+
         // Récupérer l'endpoint (safe pour les tests unitaires)
         String endpoint = "unknown";
         try {
@@ -72,6 +82,17 @@ public class GlobalExceptionMapper implements ExceptionMapper<Throwable> {
         return buildErrorResponse(exception, statusCode);
     }
 
+    /**
+     * Retourne {@code true} pour les exceptions 4xx qui représentent
+     * des cas métier attendus (ressource absente, accès refusé, session expirée).
+     * Ces exceptions ne doivent pas être loguées ERROR ni persistées.
+     */
+    private boolean isExpectedClientError(Throwable exception) {
+        return exception instanceof NotFoundException
+                || exception instanceof ForbiddenException
+                || exception instanceof NotAuthorizedException;
+    }
+
     private int determineStatusCode(Throwable exception) {
         if (exception instanceof WebApplicationException webAppException) {
             return webAppException.getResponse().getStatus();