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.

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

Cookie path bug #227

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

javaofferss
Copy link

@javaofferss javaofferss commented Feb 10, 2022

fixed #225 . cache the path=/ of proxy cookies .
The design drawing is as follows : (It has been running stably in our system.)
图片

@javaofferss
Copy link
Author

Yes I agree , I have reviewed code and test again.

@dsmiley
Copy link
Collaborator

dsmiley commented Apr 6, 2022

I don't know what the "bug" is. Maybe this is a feature? I confess I've been away from this project for quite some time and haven't looked closely at cookies before. Do you know if other proxies do this cookie path re-mapping? That's what I'd call it based on what I gleaned in your diagram but I may misunderstand.

@javaofferss
Copy link
Author

Yes, you can think of it as a feature. The purpose of this feature is to cache proxy cookies But I think this is to further improve the cookie mechanism The role of proxy cookies is not lost. The cookie of the proxy service is invalid in the previous version without modification. At present, we use the modified one, and it has been running on our server for a long time. And I designed this mechanism as optional. Only valid when docacherootpathcookies is true. The default is false

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

issue:The path of the cookie is changed. More sessions will be created for the same user
2 participants