Github user ivmaykov commented on the issue: https://github.com/apache/zookeeper/pull/679 @anmolnar the benefit of the current DOS tests is to verify that certain actions on a UnifiedSocket (getting input stream, setting socket options, etc) do not trigger the blocking read for TLS / plaintext mode detection. I think there is benefit to having such tests `UnifiedServerSocketTest`, since they test behavior that's implemented in `UnifiedServerSocket.java`. What do you think?
---