-
Notifications
You must be signed in to change notification settings - Fork 31
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
[BUG] Second CKAN Cookie in _logout Response #107
Comments
Do you have any workaround for it @OfficialOwlElder ? I might have hit the same issue (will inspect cookies to confirm), but my logs say:
but in the CKAN site I'm still logged in! 😩 |
@mixmixmix Unfortunately, I was not able to figure out where the 2nd cookie is coming from so I couldn't find a workaround. This issue has been on the back burner for me since it is only affecting the logout. I am happy to help, though this one has me stumped. |
Thanks for a prompt response @OfficialOwlElder! My issue is slightly bit different as I'm not getting two cookies (maybe because I do not get redirect back from the Identity Provider?). However I managed to be able to fix the issue by removing domain parameter:
Now when re-visiting the site after the logout, I have a new cookie and the user is logged out. I'm on |
Please note that this happened after migrating our CKAN instance from an Ubuntu 20.04 machine to a Debian 12 machine and upgrading CKAN to the latest version of 2.10.
Describe the bug
Upon logging out, the session cookie does not get deleted leaving the user still logged in. It seems that two cookies are sent in the _logout response. There is a difference in the domains, but as I understand this should not matter.
One domain starts with a dot:
.subdomain.domain.com
where as the other is normal:
subdomain.domain.com
The first Response Cookie is the expected cookie with the correct expiration date. The second cookie is being injected somewhere and replacing the original cookie. I have tried printing the all response headers in the logout() function and I only see the correct cookie:
Please let me know if I can provide any additional information.
Thank you for your support.
ckanext-saml2auth version affected
v1.1.0
Expected behaviour
Upon logging out, one ckan cookie should be included in the _logout response setting the expiration time to 0.
Logs
Everything in the logs looks normal for both ckan-worker and ckan-uwsgi.
The text was updated successfully, but these errors were encountered: