Hi,

> We really want to work with you to get this patch into OpenSSL to help 
> developers of HTTP/2.0 draft implementations. We welcome your assistance to 
> review this patch.

What really makes this patch unusable (at least for us) is this snippet:

+#if !defined(OPENSSL_NO_NEXTPROTONEG) && !defined(OPENSSL_NO_ALPN)
+#error "Cannot define both NPN and ALPN"
+#endif

We simply cannot drop support for NPN (i.e. SPDY) just to add support for ALPN.

IMHO, the correct solution would be to always prefer and offer ALPN,
unless client announced only NPN support in Client Hello.

Best regards,
Piotr Sikora
______________________________________________________________________
OpenSSL Project                                 http://www.openssl.org
Development Mailing List                       [email protected]
Automated List Manager                           [email protected]

Reply via email to