-
Notifications
You must be signed in to change notification settings - Fork 7.3k
New issue
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
修改用户时,日志保存逻辑中抛出StackOverFlow错误 #609
Comments
目前我是将 |
awesomePC
added a commit
to awesomePC/Admin
that referenced
this issue
Apr 23, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
版本
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: