Closed msteinert closed 7 years ago
@anirudhvr has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
This looks fine, though you may not want a stronger check if your application depends on ALPN/NPN support or codec selection, or want to warn when using OpenSSL without NPN/ALPN support.
Use getSelectedNextProtocolNoThrow instead of getSelectedNextProtocol so that the handshake can complete with older versions of OpenSSL that do not support protocol negotiation.