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

http - http.IncomingMessage.aborted #15260

Closed
ronag opened this issue Sep 8, 2017 · 1 comment
Closed

http - http.IncomingMessage.aborted #15260

ronag opened this issue Sep 8, 2017 · 1 comment
Labels
http Issues or PRs related to the http subsystem. http2 Issues or PRs related to the http2 subsystem.

Comments

@ronag
Copy link
Member

ronag commented Sep 8, 2017 •
edited
Loading

Maybe http.IncomingMessage should have an aborted property similar to http2.Http2Stream. At least for consisteny?
@ronag ronag changed the title http.ClientRequest.aborted http.IncomingMessage.aborted Sep 8, 2017
@ronag ronag changed the title http.IncomingMessage.aborted http - http.IncomingMessage.aborted Sep 8, 2017
@mscdex mscdex added http Issues or PRs related to the http subsystem. http2 Issues or PRs related to the http2 subsystem. labels Sep 8, 2017
@apapirovski
Copy link
Contributor

I don't think this is happening anymore given that #15270 was closed? Could you confirm @ronag? In general, I don't think there's as much of an expectation of compatibility between the two clients as there is between the two servers.

@ronag ronag closed this as completed Nov 29, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
http Issues or PRs related to the http subsystem. http2 Issues or PRs related to the http2 subsystem.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@apapirovski @mscdex @ronag