fix #2839 Adjust API response to conditionally include exception mess…

…ages based on configuration.
codelibs · Aug 15, 2024 · 17213f3 · 17213f3
1 parent 0df246e
commit 17213f3
Showing 1 changed file with 10 additions and 2 deletions.
diff --git a/src/main/java/org/codelibs/fess/api/engine/SearchEngineApiManager.java b/src/main/java/org/codelibs/fess/api/engine/SearchEngineApiManager.java
@@ -102,8 +102,16 @@ public void process(final HttpServletRequest request, final HttpServletResponse
                 throw new WebApiException(HttpServletResponse.SC_FORBIDDEN, "Invalid session.");
             });
         } catch (final WebApiException e) {
-            logger.debug("Failed to access to Web API.", e);
-            response.sendError(e.getStatusCode(), e.getMessage());
+            String message;
+            if (Constants.TRUE.equalsIgnoreCase(ComponentUtil.getFessConfig().getApiJsonResponseExceptionIncluded())) {
+                logger.warn("Failed to access to Web API.", e);
+                message = e.getMessage();
+            } else {
+                final String errorCode = UUID.randomUUID().toString();
+                message = "[" + errorCode + "] Failed to access to Web API.";
+                logger.warn(message, e);
+            }
+            response.sendError(e.getStatusCode(), message);
         }
     }