Skip to content
Permalink
Browse files

8235874: The ordering of Cipher Suites is not maintained provided thr…

…ough jdk.tls.client.cipherSuites and jdk.tls.server.cipherSuites system property

Corrected Cipher Suites ordering through system properties

Reviewed-by: xuelei
  • Loading branch information
Sibabrata Sahoo
Sibabrata Sahoo committed Dec 16, 2019
1 parent abac8b1 commit fca342f7369a56d7c51f95a3f6607eb4b2d9438f
@@ -373,7 +373,7 @@ boolean isStaplingEnabled(boolean isClient) {
private static List<CipherSuite> getApplicableCipherSuites(
Collection<CipherSuite> allowedCipherSuites,
List<ProtocolVersion> protocols) {
TreeSet<CipherSuite> suites = new TreeSet<>();
LinkedHashSet<CipherSuite> suites = new LinkedHashSet<>();
if (protocols != null && (!protocols.isEmpty())) {
for (CipherSuite suite : allowedCipherSuites) {
if (!suite.isAvailable()) {
@@ -30,7 +30,6 @@
* @library /javax/net/ssl/templates
* /javax/net/ssl/TLSCommon
* @summary Test TLS ciphersuites order set through System properties
* @ignore Not applicable until the cipher order of system property maintained.
* @run main/othervm
* -Djdk.tls.client.cipherSuites=TLS_CHACHA20_POLY1305_SHA256,TLS_AES_128_GCM_SHA256,TLS_AES_256_GCM_SHA384
* -Djdk.tls.server.cipherSuites=TLS_AES_256_GCM_SHA384,TLS_CHACHA20_POLY1305_SHA256,TLS_AES_128_GCM_SHA256

0 comments on commit fca342f

Please sign in to comment.
You can’t perform that action at this time.