-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
RESTEasy Reactive exception mapper issues #15763
Comments
/cc @geoand, @stuartwdouglas |
#15767 takes care of the second problem. The first one I can't reproduce. |
The first problem is due to Vertx event-loop threads hanging on to the original Quarkus ClassLoader. |
Actually I am pretty sure I know what the problem is but I need to investigate more |
#15779 should fix the first case |
geoand
added a commit
to geoand/quarkus
that referenced
this issue
Mar 16, 2021
geoand
added a commit
that referenced
this issue
Mar 16, 2021
Make per class @ServerExceptionMapper work with Mutiny response types
geoand
added a commit
to geoand/quarkus
that referenced
this issue
Mar 17, 2021
geoand
added a commit
that referenced
this issue
Mar 17, 2021
Make sure the proper TCCL isn't lost in dev mode when static resource exist
Thanks! |
YW! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
This one happens only in DEV mode. Start with this endpoint:
Now trigger the 404 with
GET /fruit/123
, you get the DEV Mode 404 red page.Now uncomment the exception mapper and reload the page, you get:
This looks very similar to #15748, but only for DEV mode. Perhaps it's different?
Now, a more annoying thing is that if I change the endpoint method to:
It behaves differently and doesn't even try to run my exception mapper, when it should be identical to the previous endpoint.
The text was updated successfully, but these errors were encountered: