Thanks, you beat me by 10 minutes (and your solution is nicer than mine).
Please do not wait for the docs with tagging. I have a bigger SSL docs
change in mind which I'll not finish today.
Tanks and Regards,
Rainer
Am 08.09.2017 um 15:13 schrieb ma...@apache.org:
Author: markt
Date: Fri Sep 8 13:13:42 2017
New Revision: 1807755
URL: http://svn.apache.org/viewvc?rev=1807755&view=rev
Log:
Handle possible NPE in unit test
Modified:
tomcat/trunk/test/org/apache/tomcat/util/net/openssl/TestOpenSSLConf.java
Modified:
tomcat/trunk/test/org/apache/tomcat/util/net/openssl/TestOpenSSLConf.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/tomcat/util/net/openssl/TestOpenSSLConf.java?rev=1807755&r1=1807754&r2=1807755&view=diff
==
--- tomcat/trunk/test/org/apache/tomcat/util/net/openssl/TestOpenSSLConf.java
(original)
+++ tomcat/trunk/test/org/apache/tomcat/util/net/openssl/TestOpenSSLConf.java
Fri Sep 8 13:13:42 2017
@@ -41,8 +41,8 @@ public class TestOpenSSLConf extends Tom
String protocol = tomcat.getConnector().getProtocolHandlerClassName();
// The tests are only supported for APR and OpenSSL
if (!protocol.contains("Apr")) {
-String sslImplementation =
-
tomcat.getConnector().getProperty("sslImplementationName").toString();
+String sslImplementation = String.valueOf(
+
tomcat.getConnector().getProperty("sslImplementationName"));
Assume.assumeTrue("This test is only for OpenSSL based SSL
connectors",
sslImplementation.contains("openssl"));
}
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org
