Skip to content
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.

Sign up for GitHub

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

Jump to bottom

nacos权限认证失败，直接弹出到登陆界面 #8417

Closed
wangqingchengsg opened this issue May 20, 2022 · 8 comments · Fixed by #8419
Closed

nacos权限认证失败，直接弹出到登陆界面 #8417

wangqingchengsg opened this issue May 20, 2022 · 8 comments · Fixed by #8419
Labels
area/Nacos console Related to Nacos consle kind/bug Category issues or prs related to bug.

Comments

@wangqingchengsg
Copy link

wangqingchengsg commented May 20, 2022 •
edited
Loading

image

点击后，告诉我没权限，直接弹出到登陆页面。。

nacos-server-2.1.0版本
@KomachiSion KomachiSion changed the title nacos权限bug-这种问题其实哎 nacos权限认证失败，直接弹出到登陆界面 May 20, 2022
@KomachiSion
Copy link
Collaborator

是不是因为token已经过期或者session已经过期导致？

@wangqingchengsg
Copy link
Author

登陆没有过期，只要是没得权限，直接跳转到登陆页面

@onewe
Copy link
Collaborator

onewe commented May 20, 2022

@KomachiSion 估计是前端的那个判断有点问题,后面我复现一下

@onewe
Copy link
Collaborator

onewe commented May 20, 2022

问题是这样的,之前为了解决 nacos重启session失效,用户任然能正常操作,只有在修改用户时,才提示authorization failed!. issue: #8107 当时的解决方案是: 当后端返回403时,则跳转到登录页面.

@onewe
Copy link
Collaborator

onewe commented May 20, 2022

现在又有个public 无权限问题 #8107 (comment)

@onewe
Copy link
Collaborator

onewe commented May 20, 2022

@wangqingchengsg 请问你这个是升级导致的问题还是一开始就有这个问题?

@KomachiSion
Copy link
Collaborator

问题是这样的,之前为了解决 nacos重启session失效,用户任然能正常操作,只有在修改用户时,才提示authorization failed!. issue: #8107 当时的解决方案是: 当后端返回403时,则跳转到登录页面.

如果是403就出现跳转那肯定不对啊，如果是session失效导致的权限失败应该给其他的错误码。

@onewe
Copy link
Collaborator

onewe commented May 20, 2022

@i will resolve it@

@onewe onewe mentioned this issue May 20, 2022
Merged
5 tasks
@KomachiSion KomachiSion added kind/bug Category issues or prs related to bug. area/Nacos console Related to Nacos consle and removed status/need feedback labels May 23, 2022
onewe added a commit to onewe/nacos that referenced this issue May 23, 2022
@onewe
[ISSUE alibaba#8417] throw a HttpSessionRequiredException when sessio…
5203788 
…ns expired
onewe added a commit to onewe/nacos that referenced this issue May 23, 2022
@onewe
[ISSUE alibaba#8417] add unit test
02fc0ed
onewe added a commit to onewe/nacos that referenced this issue May 23, 2022
@onewe
[ISSUE alibaba#8417] repackage
6c5cdd7 
Close alibaba#8417
onewe added a commit to onewe/nacos that referenced this issue May 23, 2022
@onewe
[ISSUE alibaba#8417] retry ci
9c005b5
onewe added a commit to onewe/nacos that referenced this issue May 23, 2022
@onewe
[ISSUE alibaba#8417] retry ci
f86e444
KomachiSion pushed a commit that referenced this issue May 23, 2022
@onewe
[ISSUES #8417] throw a HttpSessionRequiredException when sessions exp…
d6f2740 
…ired (#8419)

* [ISSUE #8417] throw a HttpSessionRequiredException when sessions expired

* [ISSUE #8417] add unit test

* [ISSUE #8417] repackage

Close #8417

* [ISSUE #8417] retry ci

* [ISSUE #8417] retry ci
@onewe onewe mentioned this issue Jul 21, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area/Nacos console Related to Nacos consle kind/bug Category issues or prs related to bug.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[ISSUES #8417] throw a HttpSessionRequiredException when sessions expired onewe/nacos
3 participants
@wangqingchengsg @KomachiSion @onewe