AOP跨模块捕获异常遭CGLIB拦截而继续向上抛出异常
前言
最近，在开发过程中，我遇到一个不易察觉的小bug。这个bug并没有直接给出报错信息，使得排查问题的根源变得困难。我希望通过分享这个经验，帮助大家避免重蹈覆辙，以免浪费不必要的时间和精力。
为了避免类似的困境，我们应当时刻保持警惕，对开发过程中的每一个细节都进行严格的检查。同时，利用调试工具和日志输出等功能，可以帮助我们更快速地定位和解决问题。此外，定期进行代码审查和测试也是非常必要的，这有助于发现潜在的问题并及时解决。
一、BUG详情
1.1 报错信息
如下图所示：
java.lang.reflect.UndeclaredThrowableException: null
at org.springframework.aop.framework.CglibAopProxy$CglibMethodInvocation.proceed(CglibAopProxy.java:780) ~[spring-aop-5.3.27.jar:5.3.27]
Caused by: exception.NoAuthorityException: 无权限访问1.2 接口响应信息
预期是抛出无权限访问异常，但是没有被aop捕获，被上层UndeclaredThrowableException异常捕获。
1.3 全局异常处理器的定义
如下图所示：
@ExceptionHandler(Exception.class)
public <T> R<T> handle(Exception exception) {
if (exception instanceof BindException) {
// Bind错误特殊处理
return R.wrap(() -> {
ApiValidationUtil.checkBinding(((BindException) exception).getBindingResult());
return null;
});
}
return R.errorAndLog(exception);
}二、排查过程
找到最上面一层报错，发现错误在CglibAopProxy.class中。
附上源码逻辑：
@Override
@Nullable
public Object proceed() throws Throwable {
try {
return super.proceed();
}
catch (RuntimeException ex) {
throw ex;
}
catch (Exception ex) {
if (ReflectionUtils.declaresException(getMethod(), ex.getClass()) ||
KotlinDetector.isKotlinType(getMethod().getDeclaringClass())) {
// Propagate original exception if declared on the target method
// (with callers expecting it). Always propagate it for Kotlin code
// since checked exceptions do not have to be explicitly declared there.
throw ex;
}
else {
// Checked exception thrown in the interceptor but not declared on the
// target method signature -> apply an UndeclaredThrowableException,
// aligned with standard JDK dynamic proxy behavior.
throw new UndeclaredThrowableException(ex);
}
}
}从源码可以看出咱们最后抛出的异常就是UndeclaredThrowableException异常，所以说if块里面的逻辑是false。
继续深挖ReflectionUtils.declaresException(getMethod(), ex.getClass())方法的逻辑。
附上declaresException方法源码：
public static boolean declaresException(Method method, Class<?> exceptionType) {
Assert.notNull(method, "Method must not be null");
Class<?>[] declaredExceptions = method.getExceptionTypes();
for (Class<?> declaredException : declaredExceptions) {
if (declaredException.isAssignableFrom(exceptionType)) {
return true;
}
}
return false;
}method就是方法体了，exceptionType就是异常类型了。
method.getExceptionTypes()从controller层读到异常类型存入declaredExceptions中，与传入的exceptionType进行判断。
declaredException.isAssignableFrom(exceptionType)的意思是declaredException是不是exceptionType的父类。只要满足捕获的异常是接口抛出异常的父类就行了。
因为原来的controller层接口是并没有声明异常。
如下所示：
//原先的接口
@Role(400)
@Override
public R<UserInfoVO> getUserInfo(String loginName) {
Assert.notNull(loginName, "请求参数为空");
return sysUserInfoService.getUserInfo(loginName);
}所以declaredExceptions是空的，那当然返回的是false。
所以走了else的逻辑，向上抛出throw new UndeclaredThrowableException(ex)。
三、解决方案
在接口方法上声明错误类型（exceptionType）。
如下所示：
@Role(400)
@Override
public R<UserInfoVO> getUserInfo(String loginName) throws NoAuthorityException {
Assert.notNull(loginName, "请求参数为空");
return sysUserInfoService.getUserInfo(loginName);
}这样的话Class<?>[] declaredExceptions = method.getExceptionTypes();可以读到NoAuthorityException 异常，并和拦截到的异常ex.getClass()得到也是NoAuthorityException异常做对比，满足isAssignableFrom方法，所以成功捕获。
由此可见，我们把ex.getClass()，也就是AOP里要捕获的异常设置为Exception也是可以满足需求的。
附一张成功响应图：
四、总结
在本次博客中，我们讨论了AOP跨模块捕获异常时，CGLIB拦截导致异常继续向上抛出的问题。通过分析问题原因和解决方案，我们了解到CGLIB拦截异常是由于代理对象与目标对象继承关系导致的问题。通过使用AspectJ的解决方案，我们可以避免该问题的发生，从而更好地实现AOP功能。
通过分析CGLIB拦截异常的原因和提出解决方案，我们更好地了解了AOP的实现方式和如何解决跨模块异常处理的问题。这对于在实际开发中更好地应用AOP技术具有重要的指导意义。