레거시 커스텀 Exception 혼재로 인한 AOP 예외 처리 및 공통 응답 구조 개선

// 개선 전: 무분별한 System.out 및 stack trace 직접 출력
try {
    businessService.execute(data);
} catch (GlobalException e) {
    System.out.println("에러 발생: " + e.getMessage());
    e.printStackTrace();
}
 
// 개선 후: Facade 기반 log4j 구조화 로깅
public class LoggingFacade {
    private static final Logger logger = Logger.getLogger(LoggingFacade.class);
 
    public static void logError(String contextMessage, Throwable throwable) {
        // 전체 Stack Trace 대신 핵심 원인 메시지 + 최상단 프레임만 정제
        String simplifiedStack = getSimplifiedStackTrace(throwable);
        logger.error(String.format("[%s] Error Message: %s | Stack Message: %s",
            contextMessage, throwable.getMessage(), simplifiedStack));
    }
 
    private static String getSimplifiedStackTrace(Throwable t) {
        if (t == null) return "No Stack Trace";
        StackTraceElement[] elements = t.getStackTrace();
        if (elements.length > 0) {
            StackTraceElement topFrame = elements[0];
            return String.format("%s.%s(Line:%d)",
                topFrame.getClassName(), topFrame.getMethodName(), topFrame.getLineNumber());
        }
        return t.getMessage();
    }
}

@RestControllerAdvice
public class GlobalExceptionHandler {
 
    @ExceptionHandler(GlobalException.class)
    public ResponseEntity<ErrorResponse> handleGlobalException(GlobalException ex) {
        ErrorCode errorCode = ex.getErrorCode(); // 예외에 매핑된 고유 코드 + HTTP Status
 
        LoggingFacade.logError("API Exception Handler Blocked", ex); // 3.1 간소화 로깅 재사용
 
        ErrorResponse response = new ErrorResponse(
            errorCode.getReturnCode(),
            ex.getMessage() != null ? ex.getMessage() : errorCode.getDefaultMessage()
        );
        return new ResponseEntity<>(response, HttpStatus.valueOf(errorCode.getHttpStatus()));
    }
 
    @ExceptionHandler(Exception.class) // 매핑되지 않은 미분류 예외의 안전망
    public ResponseEntity<ErrorResponse> handleUnexpected(Exception ex) {
        LoggingFacade.logError("Unmapped Exception", ex);
        ErrorResponse response = new ErrorResponse(ErrorCode.INTERNAL_ERROR.getReturnCode(),
            ErrorCode.INTERNAL_ERROR.getDefaultMessage());
        return new ResponseEntity<>(response, HttpStatus.INTERNAL_SERVER_ERROR);
    }
}

// ErrorCode Enum: 세분화된 returnCode(고객사 요청 흡수)는 자유롭게 늘리되,
//                 httpStatus는 4xx/5xx의 정해진 집합으로만 수렴시킨다.
public enum ErrorCode {
    // returnCode(비즈니스 식별자, N)          httpStatus(전송 신호, 1)  logLevel
    INVALID_PARAMETER("E4001", 400, Level.WARN),
    DUPLICATE_REQUEST ("E4002", 400, Level.WARN),
    POLICY_VIOLATION  ("E4090", 409, Level.WARN),
    AUTH_EXPIRED      ("E4011", 401, Level.INFO),
    NO_PERMISSION     ("E4030", 403, Level.WARN),
    EXTERNAL_TIMEOUT  ("E5020", 502, Level.ERROR),
    INTERNAL_ERROR    ("E5000", 500, Level.ERROR);
    // ... 고객사별 커스텀 예외는 여기에 returnCode만 추가, httpStatus는 위 집합 재사용
}

<error-page>
    <error-code>404</error-code>
    <location>/WEB-INF/views/error/404.jsp</location>
</error-page>
<error-page>
    <error-code>500</error-code>
    <location>/WEB-INF/views/error/500.jsp</location>
</error-page>
<error-page>
    <exception-type>com.enterprise.exception.GlobalException</exception-type>
    <location>/WEB-INF/views/error/globalError.jsp</location>
</error-page>

public class AjaxErrorFilter implements Filter {
    @Override
    public void doFilter(ServletRequest request, ServletResponse response, FilterChain chain)
            throws IOException, ServletException {
        HttpServletRequest httpRequest = (HttpServletRequest) request;
        HttpServletResponse httpResponse = (HttpServletResponse) response;
        try {
            chain.doFilter(request, response);
        } catch (Exception ex) {
            if ("XMLHttpRequest".equals(httpRequest.getHeader("X-Requested-With"))) {
                httpResponse.setContentType("application/json;charset=UTF-8");
                httpResponse.setStatus(HttpServletResponse.SC_INTERNAL_SERVER_ERROR);
                httpResponse.getWriter().write(
                    String.format("{\"success\":false,\"message\":\"%s\"}", ex.getMessage()));
            } else {
                throw ex; // 일반 브라우저 요청은 컨테이너 예외 처리기로 재전파 → 에러 페이지
            }
        }
    }
}