★ wanayoo — archive 1999 https://github.com/nodejs/node/pull/18182Nouvelle recherche | Portail wanayoo
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.

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

http2: add a number of checks for server close callback #18182

Closed
wants to merge 1 commit into from
Closed

http2: add a number of checks for server close callback #18182

wants to merge 1 commit into from

Conversation

jasnell
Copy link
Member

@jasnell jasnell commented Jan 16, 2018

Verify that server close callbacks are being called

Refs: #18176

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • commit message follows commit guidelines
Affected core subsystem(s)

http2

@jasnell
http2: add a number of checks for server close callback
a9c3f36 
Verify that server close callbacks are being called
@jasnell
Copy link
Member Author

@jasnell jasnell commented Jan 16, 2018

CI: https://ci.nodejs.org/job/node-test-pull-request/12559/

@jasnell jasnell added the http2 label Jan 16, 2018
@BridgeAR
Copy link
Member

@BridgeAR BridgeAR commented Jan 19, 2018

Landed in 0a1adc0

@BridgeAR BridgeAR closed this Jan 19, 2018
BridgeAR added a commit to BridgeAR/node that referenced this issue Jan 19, 2018
@jasnell @BridgeAR
http2: add checks for server close callback
0a1adc0 
Verify that server close callbacks are being called

PR-URL: nodejs#18182
Refs: nodejs#18176
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
evanlucas added a commit that referenced this issue Jan 30, 2018
@jasnell @evanlucas
http2: add checks for server close callback
9c16171 
Verify that server close callbacks are being called

PR-URL: #18182
Refs: #18176
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
@evanlucas evanlucas mentioned this pull request Jan 30, 2018
Merged
evanlucas added a commit that referenced this issue Jan 30, 2018
@jasnell @evanlucas
http2: add checks for server close callback
ee6217a 
Verify that server close callbacks are being called

PR-URL: #18182
Refs: #18176
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
@MylesBorins
Copy link
Member

@MylesBorins MylesBorins commented Feb 27, 2018

Should this be backported to v8.x-staging? If yes please follow the guide and raise a backport PR, if not let me know or add the dont-land-on label.

kjin added a commit to kjin/node that referenced this issue May 1, 2018
@jasnell @kjin
http2: add checks for server close callback
9ff0e8c 
Verify that server close callbacks are being called

PR-URL: nodejs#18182
Refs: nodejs#18176
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
MylesBorins added a commit that referenced this issue May 2, 2018
@jasnell @MylesBorins
http2: add checks for server close callback
ac64b4f 
Verify that server close callbacks are being called

Backport-PR-URL: #20456
PR-URL: #18182
Refs: #18176
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
@MylesBorins MylesBorins mentioned this pull request May 2, 2018
Merged
MayaLekova added a commit to MayaLekova/node that referenced this issue May 8, 2018
@jasnell @MayaLekova
http2: add checks for server close callback
f5a300a 
Verify that server close callbacks are being called

PR-URL: nodejs#18182
Refs: nodejs#18176
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@addaleax addaleax

@BridgeAR BridgeAR

Assignees
No one assigned
Labels
author ready fast-track http2 test
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

None yet
5 participants
@jasnell @BridgeAR @MylesBorins @addaleax @nodejs-github-bot