Skip to content

Commit

Permalink
Fix #78: TLS connections broken, if no explicit TLS Protocol is speci…
Browse files Browse the repository at this point in the history
…fied
  • Loading branch information
gunterze committed Apr 10, 2017
1 parent 596a80e commit 3155f30
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions dcm4che-net/src/main/java/org/dcm4che3/net/Connection.java
Original file line number Diff line number Diff line change
Expand Up @@ -1230,7 +1230,7 @@ private SSLSocket createTLSSocket(Socket s, Connection remoteConn)
SSLSocket ssl = (SSLSocket) sf.createSocket(s,
remoteConn.getHostname(), remoteConn.getPort(), true);
ssl.setEnabledProtocols(
intersect(remoteConn.tlsProtocols, tlsProtocols));
intersect(remoteConn.tlsProtocols(), tlsProtocols()));
ssl.setEnabledCipherSuites(
intersect(remoteConn.tlsCipherSuites, tlsCipherSuites));
ssl.startHandshake();
Expand All @@ -1252,8 +1252,8 @@ public boolean isCompatible(Connection remoteConn) {
if (!isTls())
return !remoteConn.isTls();

return hasCommon(remoteConn.tlsProtocols, tlsProtocols)
&& hasCommon(remoteConn.tlsCipherSuites, tlsCipherSuites);
return hasCommon(remoteConn.tlsProtocols(), tlsProtocols())
&& hasCommon(remoteConn.tlsCipherSuites, tlsCipherSuites);
}

private boolean hasCommon(String[] ss1, String[] ss2) {
Expand Down

0 comments on commit 3155f30

Please sign in to comment.