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

Fix explanation on default settings for content negotiation in reference doc #27067

Merged

Conversation

hrybs
Copy link
Contributor

@hrybs hrybs commented Jun 15, 2021

The official documentation "Web on Servlet Stack" contains paragraph "1.11.6. Content Types" that contains incorrect description of default strategy to determine the requested media types from the request.
According to source code of "org.springframework.web.servlet.config.annotation.WebMvcConfigurationSupport" the PathExtensionContentNegotiationStrategy is deprecated and is not used as a default strategy.

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged or decided on label Jun 15, 2021
@rstoyanchev rstoyanchev self-assigned this Jun 21, 2021
@rstoyanchev rstoyanchev added in: web Issues in web modules (web, webmvc, webflux, websocket) type: documentation A documentation task and removed status: waiting-for-triage An issue we've not yet triaged or decided on labels Jun 21, 2021
@rstoyanchev rstoyanchev added this to the 5.3.9 milestone Jun 21, 2021
@rstoyanchev rstoyanchev changed the title Fix content negotiation description Fix error in reference doc for content negotiation, default settings Jun 21, 2021
@rstoyanchev rstoyanchev merged commit f3f0bd2 into spring-projects:main Jul 12, 2021
@rstoyanchev rstoyanchev changed the title Fix error in reference doc for content negotiation, default settings Fix explanation on default settings for content negotiation in reference doc Jul 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in: web Issues in web modules (web, webmvc, webflux, websocket) type: documentation A documentation task
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants