Skip to content

Drop NPN support #280

Closed
Closed

Description

@florimondmanca

@pgjones suggested in #276 (comment) to stop using NPN to get the HTTP protocol in .get_http_version() on concurrency backends, and switch to ALPN-only.

(Technical notes of mine: NPN (Next Protocol Negociation) is a TLS extension that was added as part of SPDY a Google protocol whose spec eventually got merged into the HTTP/2 standard. Google announced this in 2016 in Hello HTTP/2, good bye SPDY, and ALPN now replaces NPN.)

I did a bit of research, and NPN removal does seem to have been on its way across the interwebs:

Anyone more experienced than me on HTTP protocols negociation got any thoughts on this?

Activity

  1. changed the title [-]Remove NPN verification in get_http_version()[/-] [+]Drop NPN support[/+] on Aug 26, 2019
  2. sethmlarson commented on Aug 27, 2019

    @sethmlarson
    Contributor

    You can add macOS to the list as well. The new Network module only supports ALPN.

  3. lovelydinosaur commented on Aug 27, 2019

    @lovelydinosaur
    Member

    Fair enough, that sounds like a nice little improvement.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    good first issueGood for newcomershttp/2Issues and PRs related to HTTP/2tls+pkiIssues and PRs related to TLS and PKI

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions