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

Take servlet context path into account and ensure HTTP method is preserved on redirect to requested URL #12796

Closed
wants to merge 0 commits into from
Closed

Take servlet context path into account and ensure HTTP method is preserved on redirect to requested URL #12796

wants to merge 0 commits into from

Conversation

mches
Copy link
Contributor

@mches mches commented Feb 27, 2023

  • Makes RequestedUrlRedirectInvalidSessionStrategy take servlet context path into account in putting together the redirect URL.
  • Makes the 3xx status code used by DefaultRedirectStrategy configurable.
  • Makes RequestedUrlRedirectInvalidSessionStrategy use status 307 Temporary Redirect to prevent user agents from changing the HTTP method.

I'd like to see this backported to 5.8.x at least.

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Feb 27, 2023
@mches mches changed the title Take servlet context path into account in RequestedUrlRedirectInvalidSessionStrategy Take servlet context path and HTTP method into account in redirects Feb 28, 2023
@mches mches changed the title Take servlet context path and HTTP method into account in redirects Take servlet context path into account and ensure HTTP method is preserved on redirect to requested URL Feb 28, 2023
@mches mches closed this Mar 2, 2023
@mches mches deleted the gh-12795-take-context-path-into-account branch March 2, 2023 02:49
@mches
Copy link
Contributor Author

mches commented Mar 2, 2023

Closed in favor of gh-12817

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: waiting-for-triage An issue we've not yet triaged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants