Skip to content

Conversation

@catturtle123
Copy link

@catturtle123 catturtle123 commented Jan 18, 2026

This PR addresses issue #36121

Summary

  • Improved JacksonJsonDecoder to natively support the application/*+x-ndjson media type.
  • Added tests to verify that vendor-suffixed NDJSON types (e.g., application/vnd.example.v1+x-ndjson) are correctly decoded out-of-the-box.
  • Confirmed that custom components are no longer required to handle various NDJSON media types.

Resolved Issue

  • Ensured that JacksonJsonDecoder and related codecs can handle all application/*+x-ndjson types, including vendor-specific variants, without additional configuration.
  • Prevents regressions by covering these cases in the test suite.

Integration Test

I’ve added an integration test for this behavior. If needed, I can include it in the PR as well.
스크린샷 2026-01-18 오후 2 23 56

@bclozel
Copy link
Member

bclozel commented Jan 19, 2026

Declining, see #36121 (comment)

@bclozel bclozel closed this Jan 19, 2026
@bclozel bclozel added status: declined A suggestion or change that we don't feel we should currently apply and removed status: waiting-for-triage An issue we've not yet triaged or decided on labels Jan 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: declined A suggestion or change that we don't feel we should currently apply

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants