Skip to content

Align TestRestTemplate default redirect settings to be the same as RestTemplate #43431

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

Closed
philwebb opened this issue Dec 6, 2024 · 2 comments
Assignees
Labels
type: enhancement A general enhancement
Milestone

Comments

@philwebb
Copy link
Member

philwebb commented Dec 6, 2024

The current situation is quite confusing. We use the defaults for the underlying library (usually follow redirects), unless you use HttpComponents. At that point we stop following redirects unless you create the TestRestTemplate with HttpClientOption.ENABLE_REDIRECTS.

@thecooldrop
Copy link
Contributor

Hello @philwebb if you would be available to explain what you mean by second sentence in a bit more detail I would like to start working on this issue.

@philwebb
Copy link
Member Author

Thanks for the offer @thecooldrop, but I think this one might be better handled by the team. We need to do some design work before and have some additional discussion before come up with the direction we want to take.

@philwebb philwebb added the status: pending-design-work Needs design work before any code can be developed label Feb 24, 2025
@philwebb philwebb changed the title TestRestTemplate should have same default redirect settings as RestTemplate Align TestRestTemplate default redirect settings to be the same as RestTemplate Apr 17, 2025
@philwebb philwebb modified the milestones: 3.5.x, 3.5.0-RC1 Apr 17, 2025
@philwebb philwebb removed the status: pending-design-work Needs design work before any code can be developed label Apr 17, 2025
@philwebb philwebb self-assigned this Apr 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: enhancement A general enhancement
Projects
None yet
2 participants