Skip to content
This repository has been archived by the owner on Aug 11, 2020. It is now read-only.

quic: add pending state for streams #187

Closed
wants to merge 3 commits into from
Closed

Conversation

addaleax
Copy link
Member

This resolves the situation in which openStream() is called before
the underlying native QuicSession handle exists.

Fixes: #181

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines

This resolves the situation in which `openStream()` is called before
the underlying native `QuicSession` handle exists.

Fixes: #181
addaleax added a commit that referenced this pull request Nov 1, 2019
This resolves the situation in which `openStream()` is called before
the underlying native `QuicSession` handle exists.

Fixes: #181
PR-URL: #187
Reviewed-By: James M Snell <[email protected]>
@addaleax
Copy link
Member Author

addaleax commented Nov 1, 2019

Landed in 0e86369

@addaleax addaleax closed this Nov 1, 2019
@addaleax addaleax deleted the quic-pending-stream branch November 1, 2019 12:04
addaleax added a commit that referenced this pull request Dec 11, 2019
This resolves the situation in which `openStream()` is called before
the underlying native `QuicSession` handle exists.

Fixes: #181
PR-URL: #187
Reviewed-By: James M Snell <[email protected]>
juanarbol pushed a commit to juanarbol/quic that referenced this pull request Dec 17, 2019
This resolves the situation in which `openStream()` is called before
the underlying native `QuicSession` handle exists.

Fixes: nodejs#181
PR-URL: nodejs#187
Reviewed-By: James M Snell <[email protected]>
juanarbol pushed a commit to juanarbol/quic that referenced this pull request Dec 17, 2019
This resolves the situation in which `openStream()` is called before
the underlying native `QuicSession` handle exists.

Fixes: nodejs#181
PR-URL: nodejs#187
Reviewed-By: James M Snell <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

openStream() is sync but can’t be
3 participants