-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Add the ability to set HTTP headers and Response status when returning Reactive Types from RESTEasy Reactive #20100
Comments
/cc @FroMage, @stuartwdouglas |
Hate to be pushing again - but yes, we definitely need this. Not sure if we are the first enterprise application using reactive server+client with quarkus, but this stuff is fundamental! |
We'll likely add this soon. |
Can't we return a |
I think that works, but it only covers that one use case. I'll open a POC PR for the annotations I proposed, soon |
Closes: quarkusio#20100 Co-authored-by: Yelzhas Suleimenov <[email protected]>
#20311 is what I had in mind |
Closes: quarkusio#20100 Co-authored-by: Yelzhas Suleimenov <[email protected]>
Closes: quarkusio#20100 Co-authored-by: Yelzhas Suleimenov <[email protected]>
Yes it is. Feel free to comment about that on the Pull Request |
Closes: quarkusio#20100 Co-authored-by: Yelzhas Suleimenov <[email protected]>
Closes: quarkusio#20100 Co-authored-by: Yelzhas Suleimenov <[email protected]>
Closes: quarkusio#20100 Co-authored-by: Yelzhas Suleimenov <[email protected]>
Add support for @ResponseStatus and @ResponseHeader
Description
Currently when returning a
Uni
orMulti
, there is no easy way to set a response header or a the HTTP status.We should provide and document a way to do this.
Implementation ideas
There are two ways I can think of:
@Header
and@Status
annotations that can be placed on a resource method and if the method was executed properly, these annotations set the corresponding response properties.RestResponse<Uni<T>>
andRestResponse<Multi<T>>
. This would allow more control, but might be more cumbersome to use.WDYT @FroMage @stuartwdouglas ?
The text was updated successfully, but these errors were encountered: