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.

Sign up for GitHub

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

Jump to bottom

Make sure the default content type is available in ContainerResponseFilters and WriterInterceptors when returning Response #44166

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Make sure the default content type is available in ContainerResponseFilters and WriterInterceptors when returning Response #44166

wants to merge 2 commits into from

Commits on Oct 31, 2024

  1. Use proper media type when executing writer interceptors 

    The one from the response can be null if we return a partial response
but the type is defined by @produces.
We need to consider the one from the context.

Fixes quarkusio#34839
    @gsmet
    gsmet committed Oct 31, 2024
    Configuration menu
    Copy the full SHA
    a60aca2 View commit details
    Browse the repository at this point in the history

  2. Populate the default content type even when using Response 

    Even when we use Response, it's interesting to push the default content
type to the context as it might get used by ContainerResponseFilters.

If the Response actually overrides it, it's all taken care of.

Fixes quarkusio#34839
    @gsmet
    gsmet committed Oct 31, 2024
    Configuration menu
    Copy the full SHA
    27381e3 View commit details
    Browse the repository at this point in the history