Skip to content
This repository has been archived by the owner on Dec 15, 2018. It is now read-only.

[WIP] RESTEasy module now automatically adds @Produces if required #155

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

chkal
Copy link
Contributor

@chkal chkal commented Mar 6, 2018

The specification contains the following requirement:

The semantics of controller methods differ slightly from JAX-RS resource methods; [...]. Moreover, the default media type for a response is assumed to be text/html, but otherwise can be declared using @Produces just like in JAX-RS.

This means that if a controller methods doesn't use a @Produces method to define a concrete media type, Ozark must configure the JAX-RS implementation to behave as if the controller has a @Produces("text/html) annotation.

The Jersey module already does this using a special Jersey SPI. For RESTEasy such an SPI did not exists. Therefore I filed RESTEASY-1805 and implemented this feature which is currently under review here.

So if my pull RESTEasy pull request is merged, the SPI will be available in the next RESTEasy 4.0.0 milestone. We could then update our RESTEasy dependency and apply this PR.

@chkal chkal added this to the 1.0.0-m04 milestone Mar 6, 2018
@chkal chkal self-assigned this Mar 6, 2018
@chkal
Copy link
Contributor Author

chkal commented Apr 18, 2018

A small update on this. According to the RESTEasy team, the new SPI will be included in the 3.6.0.Final release.

@chkal chkal changed the title RESTEasy module now automatically adds @Produces if required [WIP] RESTEasy module now automatically adds @Produces if required Jun 6, 2018
@chkal chkal removed this from the 1.0.0-m04 milestone Sep 23, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant