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.

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

Reflections is causing issues with Tomcat #2943

Closed
berkoab opened this issue Aug 31, 2018 · 3 comments
Closed

Reflections is causing issues with Tomcat #2943

berkoab opened this issue Aug 31, 2018 · 3 comments

Comments

@berkoab
Copy link

berkoab commented Aug 31, 2018

There is a bug in the Reflections class documented here: ronmamo/reflections#81. It is not possible to run Swagger on Tomcat 7 without receiving the error:
zip file closed
The author of Reflections has yet to create a release or release a patch for it although he has merged the fix into the main branch.

The fix is:

  1. Downgrade Reflections to 9.9.
  2. Use something else like https://github.com/classgraph/classgraph for resource scanning.
  3. Use a different branch of Reflections that has the fix in it.
@devoto13
Copy link

devoto13 commented Oct 30, 2018

The Reflections library also has several other bugs affecting Swagger users and haven't had a release for more than a year. I also would like to migrate to something more alive.

To name a few:

@re-thc
Copy link

re-thc commented Dec 30, 2018

Might be a good time now to switch to something compile time / annotation processed e.g. https://github.com/atteo/classindex ?

frantuma added a commit that referenced this issue Feb 5, 2019
ref #2943 - replace reflections with classgraph
@frantuma
Copy link
Member

frantuma commented Feb 5, 2019

#3109 addresses this by replacing reflections with classgraph

@frantuma frantuma closed this as completed Feb 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants