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

fix(Server): don't use spdy on node >= v10.0.0 #1451

Merged
merged 2 commits into from Aug 29, 2018
Merged

fix(Server): don't use spdy on node >= v10.0.0 #1451

merged 2 commits into from Aug 29, 2018

Commits on Aug 28, 2018

  1. Do not use spdy on Node 10 

    `spdy` is effectively unmaintained, and as a consequence of an
implementation that extensively relies on Node’s non-public APIs, broken
on Node 10 and above. In those cases, only https will be used for now.
Once express supports Node's built-in HTTP/2 support, migrating over to
that should be the best way to go.

Fixes: webpack#1449
Fixes: nodejs/node#21665
    @addaleax
    addaleax committed Aug 28, 2018
    Copy the full SHA
    e97d345 View commit details
    Browse the repository at this point in the history

Commits on Aug 29, 2018

  1. Merge branch 'master' into fix-node-10

    @michael-ciniawsky
    michael-ciniawsky committed Aug 29, 2018
    Copy the full SHA
    cf3374b View commit details
    Browse the repository at this point in the history