smolnar82 opened a new pull request, #1176: URL: https://github.com/apache/knox/pull/1176
[KNOX-3276:](https://issues.apache.org/jira/browse/KNOX-3276) - Fixed a bug while setting SSL protocols ## What changes were proposed in this pull request? #1175 introduced new configuration to control CM client SSL configs. Unfortunately a bug sneaked in while configuring TLS protcols in `org.apache.knox.gateway.topology.discovery.cm.DiscoveryApiClient#configureSslProtocols`: I used the `connectionSpecBuilder.cipherSuites` instead of `connectionSpecBuilder.tlsVersions` This PR fixes that bug and extends existing unit tests to cover the SSL settings too. ## How was this patch tested? Extended and ran existing unit tests. ## Integration Tests N/A ## UI changes N/A -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
