【SpringBoot】当AOP引发的异常与@RestControllerAdvice擦肩而过：异常处理的盲点揭秘

2024-01-29 15:33 由马儿跑发表于 #后端开发

各位上午/下午/晚上好呀！

今天在写bug的时候发现一个这样的问题：

AOP抛出的异常竟然没有被@RestControllerAdvice注解修饰的异常统一处理类处理。

需求是这样子滴：对某些加了自定义注解的方法进行切面处理，通过条件判断是否有权限执行该方法。

伪代码大概长这个样子：

    @Around("pointcut()")
    public Object aroundScheduledMethod(ProceedingJoinPoint joinPoint) throws Throwable {
        if (!isAccess()) {
            throw new PException();
        }
        joinPoint.proceed();
    }

其中，PException就是个Exception的子类，没什么特别的：

public class PException extends Exception {
}

在@RestControllerAdvice的统一异常处理类中处理了这个异常：

    @ExceptionHandler({PException.class})
    public Object handleAuthFailedException(PException e, HttpServletResponse response) {
        response.setStatus(HttpStatus.UNAUTHORIZED.value());
        return "Permission denied";
    }

到这里应该一切正常吧？讲道理@ExceptionHandler({PException.class})会处理AOP抛出的PException，并返回401

可是啪一跑，就不出意外的出意外了。

执行发现当权限不足时，也就是满足if条件时，主动抛出的异常并没有被@ExceptionHandler({PException.class})方法处理，而是返回了Internal Server Error。

Internal Server Error是哪里来的？别急，@RestControllerAdvice注解修饰的统一异常处理类中确实有这个，是用来捕获Exception类的，就是给整个Controller的异常处理兜底。代码如下：

    @ExceptionHandler({Exception.class})
    public Object handleException(Exception e, HttpServletResponse response) {
        response.setStatus(HttpStatus.INTERNAL_SERVER_ERROR.value());
        log.error("Internal Server Error", e);
        return "Internal Server Error";
    }

虽然我觉得上面这个东西的出现并不合理：

这个异常兜底可能会导致部分由Spring框架本身处理的异常被屏蔽掉，比如当接口返回的HttpRequestMethodNotSupportedException异常，实际上的response code是由Spring来设置成405的。如果在@RestControllerAdvice中没有处理HttpRequestMethodNotSupportedException，就会由兜底返回Internal Server Error。

不过有的项目确实希望把所有的异常控制在一定的范围内，不对外暴露，比如像下面这样：

    @ExceptionHandler({Exception.class})
    public Object handleException(Exception e, HttpServletResponse response) {
        response.setStatus(HttpStatus.BAD_REQUEST.value());
        log.error("Internal Server Error", e);
        return "Bad Request";
    }

日志中会完整的输出Exception信息，但是对外只会报错400 Bad Request来隐藏服务的异常。

扯远了，我们在这里不讨论兜底处理Exception的合理性，只讨论PException到底哪里去了，为什么没有被@ExceptionHandler({PException.class})方法处理，反而被@ExceptionHandler({Exception.class})方法处理了