Looks good to me.

Thanks,
Xuelei

On 11/28/2019 12:08 AM, [email protected] wrote:
Hi,
This simple patch just takes test javax/net/ssl/templates/SSLSocketSSLEngineTemplate.java to cover TLSv1.3.

diff -r 88502b1cf76f test/jdk/javax/net/ssl/templates/SSLSocketSSLEngineTemplate.java --- a/test/jdk/javax/net/ssl/templates/SSLSocketSSLEngineTemplate.java Mon Sep 09 11:43:16 2019 -0400 +++ b/test/jdk/javax/net/ssl/templates/SSLSocketSSLEngineTemplate.java Thu Nov 28 16:07:12 2019 +0800
@@ -1,5 +1,5 @@
  /*
- * Copyright (c) 2011, 2016, Oracle and/or its affiliates. All rights reserved. + * Copyright (c) 2011, 2019, Oracle and/or its affiliates. All rights reserved.
   * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
   *
   * This code is free software; you can redistribute it and/or modify it
@@ -124,7 +124,6 @@
      private static final String pathToStores = "../etc";
      private static final String keyStoreFile = "keystore";
      private static final String trustStoreFile = "truststore";
-    private static final String passwd = "passphrase";
      private static final String keyFilename =
              System.getProperty("test.src", ".") + "/" + pathToStores
              + "/" + keyStoreFile;
@@ -146,7 +145,7 @@
          }

          String [] protocols = new String [] {
-            "SSLv3", "TLSv1", "TLSv1.1", "TLSv1.2" };
+            "SSLv3", "TLSv1", "TLSv1.1", "TLSv1.2", "TLSv1.3" };

          for (String protocol : protocols) {
              log("Testing " + protocol);

Best regards,
John Jiang

Reply via email to