-
Notifications
You must be signed in to change notification settings - Fork 116
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.
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
Upgrade spdy
to 4.x.x and support Node v6, v8, v10
#373
Open
AdamMagaluk
wants to merge
26
commits into
master
Choose a base branch
from
spdy-upgrade
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ient. Fix issue with stream deduping when finding intenral remote port in the spdy internals.
Updates to the WS cause it to return a 400 instead of a 404 when the base path does not match.
…ribe's cb to fire a second time. Wrap cb in a once pattern. PeerSocket guard against this.ws not being initialized when closing the connection
Our tests are scheduling async operations and calling done. We need to clean this up but with the old version of mocha it was exiting the test suite after all tests were done. This enforces the old behavior.
This ensures zetta hubs running pre spdy-upgrade will be compatible at a network level. To support `h2` we need to negotiate transport and ALPN cannot help us.
Opened a new PR since i lost access to the repo after leaving Google. #379 I also published version zetta@1.6.0 that incorporates all these changes. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Well it's been about two years in making but this PR allows Zetta to run on the latest version of Node.js with the
spdy
andws
module updated to their newest versions. Sorry for all the delay, folks.This release adds support for the versions below and drops supports for node <= v4. v0.10, v0.12, etc.. Sorry for anyone who uses it but those versions are long retired. Fixes #336, #333 and will make progress with #337. Once merged we can move much fast to modernize the codebase.
Peering will work between old versions of zetta and this version and vise-versa.
Node.js Release Support (Node Releases)
v11 is going to require some work on the node-spdy front. see: spdy-http2/node-spdy#350. Or by then we can move off
node-spdy
in favor of the http2 core module.Major Changes to Spdy
ping
event from the transports Connection. spdy-http2/node-spdy#353 open to add support for receiving pings offically until then https://github.com/zettajs/node-spdy supports it.There were also changes in
ws
that broke getting the request url from the ws object. Instead we need to pass the originalrequest
to thePeerSocket
.TODO
argo
andtitan
to a version with no security vulnerabilities.