-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
io.swagger.v3.jaxrs2.Reader#resolveApplicationPath does not work with proxies #2966
Comments
Thanks for reporting this, and yes a PR is welcome! |
Ok, I will work on it and submit it ASAP since this would be useful for me. |
Turns out it's not that simple to detect if the class is a Proxy, (at least with the ones created by Weld) I'm considering checking the superclasses until I find the @ApplicationPath annotation, from my knowledge it will not be a problem, but I would like some feedback before jumping into code. |
I guess it's a good approach, shouldn't have side effects |
Hi the PR is submitted: #2974 |
Thanks! PR merged, closing ticket |
this issue is still broken w/ version |
The current implementation of resolveApplicationPath fails to find the @ApplicationPath annotation if the application is a proxy, it should search for the annotation in the superclass too.
I can provide a PR with a fix if needed.
The text was updated successfully, but these errors were encountered: