rmannibucau commented on PR #918: URL: https://github.com/apache/tomcat/pull/918#issuecomment-3572783170
@markt-asf the fix is fine for my case but note that it will break if you have cases you do use apr with graalvm (not sure it is used, never saw it there). Also think that reworking the "if" order (in this PR) is worth it since starting by testing the protocol handler is more relevant, every other tests are pointless if it is not an `AbstractHttp11Protocol` without a ssl impl and ssl enabled (so basically the issue wouldn't even trigger for not SSL cases). -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
