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

Ensure JAX-RS Request Body is properly handled when non JAX-RS annotations present #98

Merged
merged 1 commit into from
May 6, 2019

Conversation

geoand
Copy link
Contributor

@geoand geoand commented May 4, 2019

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

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
@geoand geoand changed the title Ensure JAX-RS req Body is properly handled when non JAX-RS annos present Ensure JAX-RS req Body is properly handled when non JAX-RS annotations present May 4, 2019
@geoand geoand changed the title Ensure JAX-RS req Body is properly handled when non JAX-RS annotations present Ensure JAX-RS Request Body is properly handled when non JAX-RS annotations present May 4, 2019
@kenfinnigan
Copy link
Contributor

Does this also fix #90 ?

@geoand
Copy link
Contributor Author

geoand commented May 5, 2019

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.

Copy link
Contributor

@kenfinnigan kenfinnigan left a 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?

Copy link
Contributor

@EricWittmann EricWittmann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice.

@geoand
Copy link
Contributor Author

geoand commented May 6, 2019

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

Copy link
Contributor

@msavy msavy left a 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

@EricWittmann
Copy link
Contributor

EricWittmann commented May 6, 2019

We should make sure to get #97 merged as well - before doing a release. @kenfinnigan

@EricWittmann EricWittmann merged commit fafe806 into smallrye:master May 6, 2019
@EricWittmann
Copy link
Contributor

@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! :)

@kenfinnigan
Copy link
Contributor

I've kicked off a release build

@geoand geoand deleted the jaxrs-method-params branch May 7, 2019 11:48
gasper-vrhovsek pushed a commit to gasper-vrhovsek/smallrye-open-api that referenced this pull request May 17, 2021
…lrye.config-smallrye-config-1.8.5

Bump smallrye-config from 1.8.1 to 1.8.5
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants