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

Wrong information for RequestCacheAwareFilter in the Spring Security documentation. #14855

Closed
angel-gruevski opened this issue Apr 5, 2024 · 3 comments
Labels
in: docs An issue in Documentation or samples status: ideal-for-contribution An issue that we actively are looking for someone to help us with type: bug A general bug
Milestone

Comments

@angel-gruevski
Copy link
Contributor

The documentation for Spring Security, section Servlet Architecture -> Architecture mentions the following:
"The RequestCacheAwareFilter uses the RequestCache to save the HttpServletRequest."
The statement above is incorrect, because the RequestCacheAwareFilter uses the RequestCache to get the saved HttpServletRequest after the user authenticates, while the ExceptionTranslationFilter uses the RequestCache to save the HttpServletRequest after the user requested protected resource in an unauthenticated request.

https://docs.spring.io/spring-security/reference/servlet/architecture.html#requestcacheawarefilter

@angel-gruevski angel-gruevski added status: waiting-for-triage An issue we've not yet triaged type: bug A general bug labels Apr 5, 2024
@jzheaux
Copy link
Contributor

jzheaux commented Apr 6, 2024

Fair point, @angel-gruevski. Are you able to submit a PR, correcting the documentation?

@jzheaux jzheaux added in: docs An issue in Documentation or samples status: ideal-for-contribution An issue that we actively are looking for someone to help us with and removed status: waiting-for-triage An issue we've not yet triaged labels Apr 6, 2024
@angel-gruevski
Copy link
Contributor Author

Yes, I've just submitted a PR #14860

angel-gruevski added a commit to angel-gruevski/spring-security that referenced this issue Apr 22, 2024
Correct Documentation

Corrected a mistake in the documentation related to RequestCacheAwareFilter

Closes spring-projectsgh-14855
@marcusdacoregio
Copy link
Contributor

Closing in favor of #14860

@marcusdacoregio marcusdacoregio added the status: duplicate A duplicate of another issue label Apr 30, 2024
@marcusdacoregio marcusdacoregio added this to the 5.8.13 milestone Apr 30, 2024
marcusdacoregio pushed a commit that referenced this issue Apr 30, 2024
…lter

Corrected a mistake in the documentation for the Spring Architecture where it was mentioned that RequestCacheAwareFilter is used to save the HttpServletRequest instead of the ExceptionTranslationFilter

Closes gh-14855
@marcusdacoregio marcusdacoregio removed the status: duplicate A duplicate of another issue label Apr 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in: docs An issue in Documentation or samples status: ideal-for-contribution An issue that we actively are looking for someone to help us with type: bug A general bug
Projects
None yet
Development

No branches or pull requests

3 participants