Author: markt Date: Thu Aug 25 16:49:57 2016 New Revision: 1757732 URL: http://svn.apache.org/viewvc?rev=1757732&view=rev Log: OpenSSl now disables 3DES by default so reflect this when using OpenSSL syntax to select ciphers
Modified: tomcat/tc8.0.x/trunk/java/org/apache/tomcat/util/net/jsse/openssl/OpenSSLCipherConfigurationParser.java tomcat/tc8.0.x/trunk/webapps/docs/changelog.xml Modified: tomcat/tc8.0.x/trunk/java/org/apache/tomcat/util/net/jsse/openssl/OpenSSLCipherConfigurationParser.java URL: http://svn.apache.org/viewvc/tomcat/tc8.0.x/trunk/java/org/apache/tomcat/util/net/jsse/openssl/OpenSSLCipherConfigurationParser.java?rev=1757732&r1=1757731&r2=1757732&view=diff ============================================================================== --- tomcat/tc8.0.x/trunk/java/org/apache/tomcat/util/net/jsse/openssl/OpenSSLCipherConfigurationParser.java (original) +++ tomcat/tc8.0.x/trunk/java/org/apache/tomcat/util/net/jsse/openssl/OpenSSLCipherConfigurationParser.java Thu Aug 25 16:49:57 2016 @@ -421,6 +421,10 @@ public class OpenSSLCipherConfigurationP List<Cipher> allCiphersList = Arrays.asList(Cipher.values()); Collections.reverse(allCiphersList); LinkedHashSet<Cipher> allCiphers = defaultSort(new LinkedHashSet<>(allCiphersList)); + // OpenSSL has disabled 3DES by default so remove it from the set of + // known ciphers + allCiphers.removeAll(filterByEncryption(allCiphers, Collections.singleton(Encryption.TRIPLE_DES))); + addListAlias(eNULL, filterByEncryption(allCiphers, Collections.singleton(Encryption.eNULL))); LinkedHashSet<Cipher> all = new LinkedHashSet<>(allCiphers); remove(all, eNULL); @@ -524,7 +528,7 @@ public class OpenSSLCipherConfigurationP addListAlias(SRP, filterByKeyExchange(allCiphers, Collections.singleton(KeyExchange.SRP))); initialized = true; // Despite what the OpenSSL docs say, DEFAULT also excludes SSLv2 - addListAlias(DEFAULT, parse("ALL:!EXPORT:!eNULL:!aNULL:!SSLv2:!DES:!RC2:!RC4:!DSS:!SEED:!IDEA:!CAMELLIA:!AESCCM")); + addListAlias(DEFAULT, parse("ALL:!EXPORT:!eNULL:!aNULL:!SSLv2:!DES:!RC2:!RC4:!DSS:!SEED:!IDEA:!CAMELLIA:!AESCCM:!3DES")); // COMPLEMENTOFDEFAULT is also not exactly as defined by the docs LinkedHashSet<Cipher> complementOfDefault = filterByKeyExchange(all, new HashSet<>(Arrays.asList(KeyExchange.EDH,KeyExchange.EECDH))); complementOfDefault = filterByAuthentication(complementOfDefault, Collections.singleton(Authentication.aNULL)); @@ -532,6 +536,7 @@ public class OpenSSLCipherConfigurationP complementOfDefault.addAll(aliases.get(Constants.SSL_PROTO_SSLv2)); complementOfDefault.addAll(aliases.get(EXPORT)); complementOfDefault.addAll(aliases.get(DES)); + complementOfDefault.addAll(aliases.get(TRIPLE_DES)); complementOfDefault.addAll(aliases.get(RC2)); complementOfDefault.addAll(aliases.get(RC4)); complementOfDefault.addAll(aliases.get(aDSS)); Modified: tomcat/tc8.0.x/trunk/webapps/docs/changelog.xml URL: http://svn.apache.org/viewvc/tomcat/tc8.0.x/trunk/webapps/docs/changelog.xml?rev=1757732&r1=1757731&r2=1757732&view=diff ============================================================================== --- tomcat/tc8.0.x/trunk/webapps/docs/changelog.xml (original) +++ tomcat/tc8.0.x/trunk/webapps/docs/changelog.xml Thu Aug 25 16:49:57 2016 @@ -215,6 +215,10 @@ <code>LegacyCookieProcessor</code>. Patch provided by Kyohei Nakamura. (markt) </fix> + <fix> + OpenSSl now disables 3DES by default so reflect this when using OpenSSL + syntax to select ciphers. (markt) + </fix> </changelog> </subsection> <subsection name="Jasper"> --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org