Skip to content

Conversation

@haydenrear
Copy link
Contributor

Adds ability to add message converter at specific index to builders, and propagates through to java.util.List.add(idx, ...)

    Adds ability to add message converter at specific index to builders, and propagates through to java.util.List.add(idx, ...)

Signed-off-by: hayden.rear <hayden.rear@gmail.com>
@haydenrear
Copy link
Contributor Author

@bclozel I created the PR again and got the DCO to pass as a check. Not sure if you still wanted to close it. I made a mistake on the DCO app when I installed it - it's the first time I'd used DCO.

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged or decided on label Dec 30, 2025
@bclozel bclozel changed the title Closes gh-36043 Configure HttpMessageConverters as a list Jan 7, 2026
@bclozel bclozel self-assigned this Jan 7, 2026
@bclozel bclozel added in: web Issues in web modules (web, webmvc, webflux, websocket) type: enhancement A general enhancement and removed status: waiting-for-triage An issue we've not yet triaged or decided on labels Jan 7, 2026
bclozel pushed a commit that referenced this pull request Jan 7, 2026
This commit adds a new `configureMessageConvertersList` method on the
builder to add/remove/move converters in the resulting list before they
are individually post-processed.

This allows to re-introduce a behavior that was missing with the new
contract: the ability to append a converter at the end of the list.

See gh-36083

Signed-off-by: hayden.rear <hayden.rear@gmail.com>
@bclozel bclozel closed this in 9ef4ceb Jan 7, 2026
@bclozel
Copy link
Member

bclozel commented Jan 7, 2026

Thanks for your contribution @haydenrear this is now merged in the main branch.

@bclozel bclozel added this to the 7.0.3 milestone Jan 7, 2026
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: enhancement A general enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants