-
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
MP Rest Client - meaningless exceptions #22090
Labels
Milestone
Comments
geoand
added a commit
to geoand/quarkus
that referenced
this issue
Dec 10, 2021
We now add the exact method that caused the exception in order to make it easier for users to identity where the exception is coming from Fixes: quarkusio#22090
geoand
added a commit
to geoand/quarkus
that referenced
this issue
Dec 10, 2021
We now add the exact method that caused the exception in order to make it easier for users to identity where the exception is coming from Fixes: quarkusio#22090
geoand
added a commit
to geoand/quarkus
that referenced
this issue
Dec 10, 2021
We now add the exact method that caused the exception in order to make it easier for users to identity where the exception is coming from Fixes: quarkusio#22090
geoand
added a commit
to geoand/quarkus
that referenced
this issue
Dec 10, 2021
We now add the exact method that caused the exception in order to make it easier for users to identity where the exception is coming from Fixes: quarkusio#22090
geoand
added a commit
to geoand/quarkus
that referenced
this issue
Dec 10, 2021
We now add the exact method that caused the exception in order to make it easier for users to identity where the exception is coming from Fixes: quarkusio#22090
geoand
added a commit
to geoand/quarkus
that referenced
this issue
Dec 10, 2021
We now add the exact method that caused the exception in order to make it easier for users to identity where the exception is coming from Fixes: quarkusio#22090
geoand
added a commit
that referenced
this issue
Dec 11, 2021
Provide context for Reactive Rest Client exceptions
gsmet
pushed a commit
to gsmet/quarkus
that referenced
this issue
Dec 13, 2021
We now add the exact method that caused the exception in order to make it easier for users to identity where the exception is coming from Fixes: quarkusio#22090 (cherry picked from commit 903eb34)
gsmet
pushed a commit
to gsmet/quarkus
that referenced
this issue
Dec 14, 2021
We now add the exact method that caused the exception in order to make it easier for users to identity where the exception is coming from Fixes: quarkusio#22090 (cherry picked from commit 903eb34)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Describe the bug
I'm using MicroProfile Rest Client with Quarkus and I have an issue with rest client exceptions.
My use case is simple.
If I make mistake in rest client URL I'll get 404 and following stacktrace
This stacktrace is totally meaningless.
After reading this stacktrace I don't know:
MyRestClient
MyRestClient.apiCall() or https://my-api:8080/endpoint
MyResource.callAPI() -> MySerivice.apiCall() -> MyRestClient.apiCall()
With ExceptionMapper I can extract rest client endpoint details from ClientWebApplicationException. Unfortunately I can't find any details about the call side.
How can I extract full stacktrace of the call side like "MyResource.callAPI() -> MySerivice.apiCall() -> MyRestClient.apiCall()".
I've tried creating my own ResponseExceptionMapper but I can't find nothing about the call side there.
At the moment only try-catch at the call side can give me full details about rest client call side.
Expected behavior
Quarkus should provide tdetails about rest client exception in stacktace out of the box - it's rather basic use case
Actual behavior
No response
How to Reproduce?
No response
Output of
uname -a
orver
No response
Output of
java -version
No response
GraalVM version (if different from Java)
No response
Quarkus version or git rev
No response
Build tool (ie. output of
mvnw --version
orgradlew --version
)No response
Additional information
No response
The text was updated successfully, but these errors were encountered: