markt-asf commented on pull request #461: URL: https://github.com/apache/tomcat/pull/461#issuecomment-987789405
All those references you provided are for `org.apache.commons.codec.binary.BaseNCodec` not `org.apache.tomcat.util.codec.binary.BaseNCodec` That the class is public is irrelevant. It has to be public as it is a utility class used by various Tomcat packages. Tomcat's public API is documented in the RELEASE-NOTES. The change won't be applied because it isn't necessary and there is no desire to add unnecessary code bloat to Tomcat. It your application wants to use Commons Codec then it should package it in WEB-INF/lib. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org
