search

Netflix / netflix-commons

Common utilities for Netflix OSS projects
Apache License 2.0
162 stars 81 forks source link

remove unnecessary dependency on servlet-api #24

Closed brharrington closed 1 week ago

brharrington commented 6 years ago

I noticed some of our apps that do not use servlets in any way still had this on the classpath as a transitive of eureka-client. It does not appear to actually be used by netflix-infix.

joan38 commented 5 years ago

Can we merge this @brharrington?

brharrington commented 5 years ago

It is fine with me, but I don't own this project. My understanding is that the main issue is the project is not currently in a state where it can easily be built and released. So even if merged someone would need to update it to be able to do another release.

joan38 commented 5 years ago

I saw with @elandau, he told me this project is legacy and shouldn't be used in the first place.

iuliiasobolevska commented 1 week ago

Removed in https://github.com/Netflix/netflix-commons/pull/37.