search

EalenXie / aop-log

项目正式命名为aop-log，基于Spring AOP，ThreadLocal实现方法埋点信息收集与处理。
Apache License 2.0
116 stars 54 forks source link

Bug提交 #17

Closed sivern closed 2 years ago

sivern commented 2 years ago

AopLogProcessor中 if (!aopLog.isLogOnErr() || !data.isSuccess()) 该判断是否是一个bug，理论上应该是 if (!aopLog.isLogOnErr() && !data.isSuccess()) 的关系

sivern commented 2 years ago

该bug会使配置只在异常时打印日志的配置项失效，已验证

EalenXie commented 2 years ago

十分感谢指点，但不是bug，logOnErr==true 代表仅当发生异常时才记录，!aopLog.isLogOnErr()等同于logOnErr == false，即无论是否发生异常都会记录，此条件不成立时(即logOnErr ==true时) ，仅仅只需要判断 data.isSuccess()==false即可，故判断条件为!aopLog.isLogOnErr() || !data.isSuccess()