-
Notifications
You must be signed in to change notification settings - Fork 91
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
Ensure JAX-RS Request Body is properly handled when non JAX-RS annotations present #98
Conversation
Prior to this fix, the implementation would disregard a proper JAX-RS request body parameter if it was annotated with any annotation, like for example javax.validation.Valid
Does this also fix #90 ? |
I hadn't seen that issue at all! It seems exactly the same as the Quarkus one which I tested against with this patch (and reference in the description) , so I would say yes, it does. |
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.
Seems ok to me. @EricWittmann and @msavy?
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.
Nice.
Thanks! Would it be possible to get a new release so we can include it in Quarkus and close a couple related issues that people are having? Thanks again |
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.
Looks good to me
We should make sure to get #97 merged as well - before doing a release. @kenfinnigan |
@kenfinnigan I've merged both outstanding PRs - if you can kick off a release. Should it be a standard release or a snapshot release? Not sure what we support here. Doesn't matter to me! :) |
I've kicked off a release build |
…lrye.config-smallrye-config-1.8.5 Bump smallrye-config from 1.8.1 to 1.8.5
Prior to this fix, the implementation would disregard a proper JAX-RS
request body parameter if it was annotated with any annotation, like for
example
javax.validation.Valid
Relates to: quarkusio/quarkus#2339
cc @kenfinnigan