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

Add request cache that uses cookie #8034

Closed
eleftherias opened this issue Feb 28, 2020 · 3 comments · Fixed by #8653
Closed

Add request cache that uses cookie #8034

eleftherias opened this issue Feb 28, 2020 · 3 comments · Fixed by #8653
Assignees
Labels
in: web An issue in web modules (web, webmvc) type: enhancement A general enhancement
Milestone

Comments

@eleftherias
Copy link
Contributor

Summary

Currently we have the ability to store a user's initial request using HttpSessionRequestCache. This causes the HttpSession to be read on every request.
We should provide an alternative that stores the initial request in a cookie instead of in the HttpSession.

@eleftherias eleftherias added in: web An issue in web modules (web, webmvc) type: enhancement A general enhancement labels Feb 28, 2020
@eleftherias eleftherias self-assigned this Feb 28, 2020
@zeeshanadnan
Copy link
Contributor

Hi @eleftherias hope you are doing well in these tough times. If no one is working on this issue i would like to give it a try.

@eleftherias
Copy link
Contributor Author

Thanks @zeeshanadnan! The issue is yours.

@zeeshanadnan
Copy link
Contributor

@eleftherias sorry for being late. I have submitted a pull request.

zeeshanadnan added a commit to zeeshanadnan/spring-security that referenced this issue Jul 1, 2020
eleftherias pushed a commit that referenced this issue Jul 2, 2020
@eleftherias eleftherias added this to the 5.4.0-RC1 milestone Jul 2, 2020
eleftherias added a commit that referenced this issue Jul 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in: web An issue in web modules (web, webmvc) type: enhancement A general enhancement
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants