Skip to content

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

Merged
rstoyanchev merged 1 commit intospring-projects:mainfrom
hrybs:Fix-content-negotiation-description
Jul 12, 2021
Merged

Fix explanation on default settings for content negotiation in reference doc#27067
rstoyanchev merged 1 commit intospring-projects:mainfrom
hrybs:Fix-content-negotiation-description

Conversation

@hrybs
Copy link
Copy Markdown
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.

3 participants