[GEODE-8070] rework handling of "any" in SSLUtils

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.14.0
• Component/s: membership
• Assignee: Bruce J Schuchardt
• Reporter: Bruce J Schuchardt
• Created: 05/May/20 15:33
• Updated: 03/Sep/21 02:20
• Resolved: 11/May/20 20:42

Description

SSLUtil has a list of "known" TLS protocols.  It should support TLSv1.3.

 

// lookup known algorithms
String[] knownAlgorithms = {"SSL", "SSLv2", "SSLv3", "TLS", "TLSv1", "TLSv1.1", "TLSv1.2"};
for (String algo : knownAlgorithms) {
  try {
    sslContext = SSLContext.getInstance(algo);
    break;
  } catch (NoSuchAlgorithmException e) {
    // continue
  }
} 

We probably can't fully test this change since not all JDKs we test with support v1.3 at this time.

Issue Links

• links to: GitHub Pull Request #5075