fix: can't update user info to db , fix issue:#7141 #7252
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Please do not create a Pull Request without creating an issue first.
What is the purpose of the change
修复 #7141 的问题
后台管理系统登录后, 在NacosAuthManager.login 方法中, 将登录的用户信息存于 HttpServletRequest.setAttribute 。
此种方式在ajax异步请求时,
RequestUtil.getUser
不能成功的访问到用户信息, 从而不能记录用户更新信息更换为
req.getSession().setAttribute
Brief changelog
将用户数据从 HttpServletRequest 中迁移到 HttpSession 中进行保存
Verifying this change
已验证在修改和删除时能成功记录
Follow this checklist to help us incorporate your contribution quickly and easily:
[ISSUE #123] Fix UnknownException when host config not exist
. Each commit in the pull request should have a meaningful subject line and body.mvn -B clean package apache-rat:check findbugs:findbugs -Dmaven.test.skip=true
to make sure basic checks pass. Runmvn clean install -DskipITs
to make sure unit-test pass. Runmvn clean test-compile failsafe:integration-test
to make sure integration-test pass.