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.

Sign up for GitHub

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

Jump to bottom

Reactive Routes result are not registered for reflection #10893

Closed
cescoffier opened this issue Jul 22, 2020 · 1 comment · Fixed by #10992
Closed

Reactive Routes result are not registered for reflection #10893

cescoffier opened this issue Jul 22, 2020 · 1 comment · Fixed by #10992
Assignees
@mkouba
Labels
area/reactive kind/bug Something isn't working
Milestone
1.7.0.CR1

Comments

@cescoffier
Copy link
Member

When a reactive route returns:

  • any non-string, non-buffer, non-Uni, non-Multi return type - the class must be registered for reflection
  • any Uni or Multi of non-string, non-buffer - the class must be registered for reflection (T in Uni<T>
@cescoffier cescoffier added kind/bug Something isn't working area/reactive labels Jul 22, 2020
@mkouba mkouba self-assigned this Jul 27, 2020
@mkouba mkouba added this to the 1.7.0 - master milestone Jul 27, 2020
@mkouba
Copy link
Contributor

mkouba commented Jul 27, 2020

Hm, we should also process a non-string, non-buffer method parameter annotated with @Body.

mkouba added a commit to mkouba/quarkus that referenced this issue Jul 27, 2020
@mkouba
Reactive routes - register return/body types for reflection if needed
d80360d 
- resolves quarkusio#10893
@mkouba mkouba mentioned this issue Jul 27, 2020
Merged
mkouba added a commit to mkouba/quarkus that referenced this issue Jul 27, 2020
@mkouba
Reactive routes - register return/body types for reflection if needed
bb609ec 
- resolves quarkusio#10893
mkouba added a commit to mkouba/quarkus that referenced this issue Jul 27, 2020
@mkouba
Reactive routes - register return/body types for reflection if needed
d23ca45 
- resolves quarkusio#10893
mkouba added a commit to mkouba/quarkus that referenced this issue Jul 28, 2020
@mkouba
Reactive routes - register return/body types for reflection if needed
ce5272e 
- resolves quarkusio#10893
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mkouba mkouba

Labels
area/reactive kind/bug Something isn't working
Projects
None yet
Milestone
1.7.0.CR1
Development

Successfully merging a pull request may close this issue.

Reactive routes - register return/body types for reflection if needed mkouba/quarkus
2 participants
@cescoffier @mkouba