-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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.
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
Introduce JSON by default support RESTEasy Reactive #14167
Conversation
<dependency> | ||
<groupId>io.quarkus</groupId> | ||
<artifactId>quarkus-resteasy-reactive-jsonb-deployment</artifactId> | ||
<scope>test</scope> | ||
</dependency> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was never needed in the first place
@@ -21,7 +21,6 @@ | |||
|
|||
@GET | |||
@Path("/person") | |||
@Produces(MediaType.APPLICATION_JSON) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The PR doesn't introduce any new tests, it just reuses the existing ones and removes @Produces(MediaType.APPLICATION_JSON)
to ensure everything works properly
7ba5ac5
to
e0f22a2
Compare
I have also already updated the testsuite to use the new property |
e0f22a2
to
23aae2d
Compare
return applyAdditionalDefaults(nonAsyncReturnType); | ||
} | ||
|
||
protected String[] applyAdditionalDefaults(Type nonAsyncReturnType) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done like this because I only wanted to touch default for the server endpoints
This is done when the jsonb or jackson extensions are used and only applies to application classes and the most common collection classes
23aae2d
to
ba32921
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, but this is a temporary implementation, with a more comprehensive/pluggable implementation to be added later.
This is done when the JSON-B or Jackson extensions are used
and only applies to application classes and the most common
collection classes.
This is probably more complex than absolutely necessary, but it introduces a general way
to determine default produces Media Types.