Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

tls, crypto: add ALPN Support #2564

Closed
wants to merge 2 commits into from
Closed

tls, crypto: add ALPN Support #2564

wants to merge 2 commits into from

Commits on Oct 26, 2015

  1. tls, crypto: add ALPN Support 

    ALPN is added to tls according to RFC7301, which supersedes NPN.
When the server receives both NPN and ALPN extensions from the client,
ALPN takes precedence over NPN and the server does not send NPN
extension to the client. alpnProtocol in TLSSocket always returns
false when no selected protocol exists by ALPN.
In https server, http/1.1 token is always set when no
options.ALPNProtocols exists.

PR-URL: nodejs#2564
Reviewed-By: Fedor Indutny <fedor@indutny.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
    Shigeki Ohtsu committed Oct 26, 2015
    Configuration menu
    Copy the full SHA
    7a53edd View commit details
    Browse the repository at this point in the history

  2. tls,crypto: move NPN protcol data to hidden value 

    This fix is to be consistent implementation with ALPN. Tow NPN
protocol data in the persistent memebers move to hidden variables in
the wrap object.
    Shigeki Ohtsu committed Oct 26, 2015
    Configuration menu
    Copy the full SHA
    17e560a View commit details
    Browse the repository at this point in the history