Skip to content

This re-applies Gábor's stream closing work with some additional fixes #84

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

Closed
snorp wants to merge 6 commits into from
Closed

This re-applies Gábor's stream closing work with some additional fixes #84

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
9d4279e
Proxy the 'listening' event from the underlying server
snorp Jul 16, 2014
a147f2f
Un-revert the close stream work that was reverted in node-http2-protocol
snorp Aug 26, 2014
82cdba5
Revert "Revert "HTTP - forwarding error and close events from streams…
snorp Aug 26, 2014
8ae50db
Add tests for response closing when the socket goes away
snorp Aug 28, 2014
74232ec
Don't try to write into closed streams
snorp Aug 28, 2014
48d2ea4
Fix 'close' event behavior
snorp Aug 28, 2014
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
Proxy the 'listening' event from the underlying server
  • Loading branch information
@snorp
snorp committed Aug 28, 2014
commit 9d4279ef4d7dd656126c17f3e4a3fa5060c36f7f
1 change: 1 addition & 0 deletions lib/http.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -423,6 +423,7 @@ function Server(options) {
}
});
this._server.on('request', this.emit.bind(this, 'request'));
this._server.on('listening', this.emit.bind(this, 'listening'));
}

// HTTP2 over plain TCP
Expand Down