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.

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

warning about missing json when just using resteasy or resteasy-reactive #22970

Closed
maxandersen opened this issue Jan 18, 2022 · 2 comments · Fixed by #22980
Closed

warning about missing json when just using resteasy or resteasy-reactive #22970

maxandersen opened this issue Jan 18, 2022 · 2 comments · Fixed by #22980
Assignees
Labels
area/jbang Issues related to when using jbang.dev with Quarkus area/resteasy-reactive kind/bug Something isn't working
Milestone

Comments

@maxandersen
Copy link
Contributor

Describe the bug

when just using resteasy or resteasy-classic you should get a warning that you are missing json support according to this code:

https://github.com/quarkusio/quarkus/blob/main/extensions/resteasy-classic/resteasy-common/deployment/src/main/java/io/quarkus/resteasy/common/deployment/ResteasyCommonProcessor.java#L220

But I get nothing. neither when using maven,gradle or jbang.

Attached is a resteasy classic reproducer. if change it to use resteasy-jackson I get json, but without it I get string and not warning.

resteasy.zip

Expected behavior

print during build of missing json provider.

Actual behavior

No response

How to Reproduce?

No response

Output of uname -a or ver

No response

Output of java -version

No response

GraalVM version (if different from Java)

No response

Quarkus version or git rev

No response

Build tool (ie. output of mvnw --version or gradlew --version)

No response

Additional information

No response

@maxandersen maxandersen added the kind/bug Something isn't working label Jan 18, 2022
@quarkus-bot quarkus-bot bot added area/jbang Issues related to when using jbang.dev with Quarkus area/resteasy-reactive labels Jan 18, 2022
@quarkus-bot
Copy link

quarkus-bot bot commented Jan 18, 2022

/cc @FroMage, @geoand, @quarkusio/devtools, @stuartwdouglas

@geoand
Copy link
Contributor

geoand commented Jan 18, 2022

#22980 introduces the warning for RESTEasy Reactive

geoand added a commit to geoand/quarkus that referenced this issue Jan 18, 2022
geoand added a commit to geoand/quarkus that referenced this issue Jan 19, 2022
geoand added a commit to geoand/quarkus that referenced this issue Jan 19, 2022
geoand added a commit that referenced this issue Jan 20, 2022
Warn when Resource method returns JSON but no JSON provider exists
@quarkus-bot quarkus-bot bot added this to the 2.8 - main milestone Jan 20, 2022
@gsmet gsmet modified the milestones: 2.8 - main, 2.7.0.Final Jan 21, 2022
gsmet pushed a commit to gsmet/quarkus that referenced this issue Jan 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/jbang Issues related to when using jbang.dev with Quarkus area/resteasy-reactive kind/bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants