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

Remove servlet-api from dependencies #10838

Open

2 tasks

inodb opened this issue Jun 18, 2024 · 1 comment · May be fixed by #10844

Open

2 tasks

Remove servlet-api from dependencies #10838

inodb opened this issue Jun 18, 2024 · 1 comment · May be fixed by #10844

Labels

dependencies devops good first issue

Member

inodb commented Jun 18, 2024

See: https://github.com/cBioPortal/cbioportal-docker-compose/pull/23/files#diff-e45e45baeda1c1e73482975a664062aa56f20c03dd9d64a827aba57775bed0d3R21-R22

TODO:

Confirm servlet-api is still included as dependency
If so, remove it

inodb added good first issue dependencies devops labels

dippindots linked a pull request

that will close this issue

Remove servlet-api dependency from mockserver-client-java #10844

Open

matthead commented Jul 24, 2024

I believe this issue can be closed. I am only seeing javax.servlet-api as a dependency from the mockserver jar.

[INFO] +- org.mock-server:mockserver-client-java:jar:5.15.0:test
[INFO] |  +- org.mock-server:mockserver-core:jar:5.15.0:test
[INFO] |  |  +- com.lmax:disruptor:jar:3.4.4:test
[INFO] |  |  +- javax.servlet:javax.servlet-api:jar:4.0.1:test

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment