You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ma...@apache.org on 2020/01/15 12:21:57 UTC
[tomcat] 07/07: Skip setting attribute that is neither supported
nor required for APR
This is an automated email from the ASF dual-hosted git repository.
markt pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/tomcat.git
commit 9b328a648cbcd1e2a982710465e347003e32c96b
Author: Mark Thomas <ma...@apache.org>
AuthorDate: Wed Jan 15 12:01:45 2020 +0000
Skip setting attribute that is neither supported nor required for APR
---
test/org/apache/tomcat/util/net/TestSSLHostConfigCompat.java | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/test/org/apache/tomcat/util/net/TestSSLHostConfigCompat.java b/test/org/apache/tomcat/util/net/TestSSLHostConfigCompat.java
index 90c160f..e098b54 100644
--- a/test/org/apache/tomcat/util/net/TestSSLHostConfigCompat.java
+++ b/test/org/apache/tomcat/util/net/TestSSLHostConfigCompat.java
@@ -323,7 +323,10 @@ public class TestSSLHostConfigCompat extends TomcatBaseTest {
connector.setScheme("https");
connector.setSecure(true);
Assert.assertTrue(connector.setProperty("SSLEnabled", "true"));
- Assert.assertTrue(connector.setProperty("sslImplementationName", sslImplementationName));
+ if (!connector.getProtocolHandlerClassName().contains("Apr")) {
+ // Skip this for APR. It is not supported.
+ Assert.assertTrue(connector.setProperty("sslImplementationName", sslImplementationName));
+ }
sslHostConfig.setProtocols("TLSv1.2");
connector.addSslHostConfig(sslHostConfig);
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org