for kubernetes-client#871 add TODO for tls verion detection

mlbiam · mlbiam · commit 55404aabeed0 · 2020-02-25T15:32:43.000-05:00
diff --git a/util/src/main/java/io/kubernetes/client/util/authenticators/OpenIDConnectAuthenticator.java b/util/src/main/java/io/kubernetes/client/util/authenticators/OpenIDConnectAuthenticator.java
@@ -33,6 +33,7 @@
 import java.util.Scanner;
 import javax.net.ssl.HttpsURLConnection;
 import javax.net.ssl.SSLContext;
+import javax.net.ssl.SSLEngine;
 import javax.net.ssl.TrustManagerFactory;
 import org.jose4j.json.internal.json_simple.JSONObject;
 import org.jose4j.json.internal.json_simple.parser.JSONParser;
@@ -139,6 +140,8 @@ public Map<String, Object> refresh(Map<String, Object> config) {
         TrustManagerFactory tmf = TrustManagerFactory.getInstance("PKIX");
         tmf.init(ks);
 
+        // TODO would be good to make this more dyanamic.  Doesn't seem like
+        // a good way to do this.
         sslContext = SSLContext.getInstance("TLSv1.2");
         sslContext.init(null, tmf.getTrustManagers(), new SecureRandom());
 
