We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
v2.4
本地/线上
LogAspect.logAround()
LogServiceImpl.save()
jointPoint.getArgs()
getParameter
JSONUtil.toJsonStr(argList)
curTime
logAround()
remove()
Log log = new Log("ERROR",System.currentTimeMillis() - currentTime.get());
辛苦了
The text was updated successfully, but these errors were encountered:
目前我是将getParameter()的逻辑前提到了joinPoint.proceed()之前,暂时解决了这个问题。
getParameter()
joinPoint.proceed()
Sorry, something went wrong.
elunez/eladmin-web@14aab90
[代码完善](master): 修改角色时将角色的菜单临时清空,避免日志入库数据过长
23f3b55
close elunez/eladmin#609
21853a6
No branches or pull requests
版本
v2.4
环境
本地/线上
复现步骤
问题描述
LogAspect.logAround()
里执行LogServiceImpl.save()
方法的jointPoint.getArgs()
时,获取的方法参数内容过长(我打印了参数内容,非常多roles、menus和users之间的嵌套),导致getParameter
里的JSONUtil.toJsonStr(argList)
抛出爆栈错误。LogServiceImpl.save()
抛出异常,而curTime
在logAround()
里已经被提前remove()
,导致Log log = new Log("ERROR",System.currentTimeMillis() - currentTime.get());
会有NPE异常。建议将remove()放在后面。辛苦了
The text was updated successfully, but these errors were encountered: