-
Notifications
You must be signed in to change notification settings - Fork 9k
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
Authorize button: logout is not clearing credentials #6049
Comments
I'm seeing something perhaps similar, but I'm not certain if it is caused by the same underlying issue. If it isn't, I can open up a separate issue for it. Here is what I'm seeing: I recently upgraded from v3.23.0 to v3.36.2. I use a config.json file to reference a v2 API spec and a v3 API spec document from 2 different locations, with the v3 being the default. When I load up the site everything renders fine and the oauth endpoint and scopes are correct for the v3 spec. When I select the drop down menu to switch to the v2 spec everything on the page renders correctly for the v2 spec, but if you look at the Authorize button, the oauth token endpoint and the oauth scopes are those still for the v3 spec, they did not get updated. If I refresh the whole page the Authorize button will then load the correct information. If I switch back to the v3 spec the same thing happens in reverse and I have to refresh the page again to get the correct oauth info displayed. Is this caching issue referenced above caused by the same thing causing my issue? |
same for me, logout button doesn't clear cookies, i need to do it manually |
With SpringBoot and Keycloak I just fixed it in keycloakSecurityAdapter |
For cookie clearing, I raised a PR a while ago, but no one's merging it: #9186 |
I second @sudiptosarkar, please can this be fixed and merged. When persistAuthorization is set to false there still is the ability for a session window that has not been closed to have the logout button clicked and the previous users credentials are populated and then by clicking "Authorize" again you will be logged in with the previous users credentials. This is a security issue and the correct behavior her would be to only not clear those credential fields if persistAuthorization was enabled for a given dev\test site. The current behavior is not sufficient for in production scenarios and these credential fields should be always cleared by default |
Q&A (please complete the following information)
Content & configuration
Example Swagger/OpenAPI definition:
Describe the bug you're encountering
During the logout from Authorize button, I see that "curl" doesn't contain the header with Authorize, but it seems credentials still exist in session.
To reproduce...
Steps to reproduce the behavior:
The scenario for basic auth:
Note:
The text was updated successfully, but these errors were encountered: