Skip to content

Conversation

lpinca
Copy link
Member

@lpinca lpinca commented Oct 29, 2021

Add missing 'error' event handler.

Fixes: #40610

Add missing `'error'` event handler.

Fixes: nodejs#40610
@nodejs-github-bot nodejs-github-bot added needs-ci PRs that need a full CI run. test Issues and PRs related to the tests. labels Oct 29, 2021
@lpinca
Copy link
Member Author

lpinca commented Oct 29, 2021

This fixes the issue on my machine but I'm not sure if it invalidates the test.

cc: @nodejs/http2 @kumarak

@Mesteery Mesteery added the http2 Issues or PRs related to the http2 subsystem. label Oct 29, 2021
@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

Copy link
Member

@mcollina mcollina left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@lpinca lpinca added the commit-queue Add this label to land a pull request using GitHub Actions. label Nov 4, 2021
@github-actions github-actions bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Nov 4, 2021
@github-actions
Copy link
Contributor

github-actions bot commented Nov 4, 2021

Landed in 8fce09e...0a62026

@github-actions github-actions bot closed this Nov 4, 2021
nodejs-github-bot pushed a commit that referenced this pull request Nov 4, 2021
Add missing `'error'` event handler.

Fixes: #40610

PR-URL: #40659
Reviewed-By: Matteo Collina <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
@lpinca lpinca deleted the deflake/http2-cancel-while-client-reading branch November 4, 2021 19:33
targos pushed a commit that referenced this pull request Nov 6, 2021
Add missing `'error'` event handler.

Fixes: #40610

PR-URL: #40659
Reviewed-By: Matteo Collina <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
@targos targos mentioned this pull request Nov 8, 2021
BethGriggs pushed a commit that referenced this pull request Nov 25, 2021
Add missing `'error'` event handler.

Fixes: #40610

PR-URL: #40659
Reviewed-By: Matteo Collina <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
@BethGriggs BethGriggs mentioned this pull request Nov 26, 2021
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
http2 Issues or PRs related to the http2 subsystem. needs-ci PRs that need a full CI run. test Issues and PRs related to the tests.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Investigate flaky test - http2-cancel-while-client-reading
6 participants