kirklund commented on a change in pull request #7475:
URL: https://github.com/apache/geode/pull/7475#discussion_r833659317
##########
File path:
geode-core/src/main/java/org/apache/geode/internal/net/SocketCreator.java
##########
@@ -599,10 +598,6 @@ void configureClientSSLSocket(Socket socket, HostAndPort
addr, int timeout) thro
} else {
throw ex;
}
- } catch (SSLHandshakeException ex) {
Review comment:
I've been thinking about the abundance of error handling in Geode that
checks and branches based on brittle exception messages that can change from
release to release, vendor to vendor, etc.
We may want to consider doing some searches to see how much code we're
talking about and then see what we can do to centralize as much of it as
possible so that it's easier to find and figure out what has changed in a JVM
release.
--
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]
