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

Define the contents of QPACK Encoder and Decoder streams #1361

Closed
martinthomson opened this issue May 21, 2018 · 1 comment · Fixed by #1467
Closed

Define the contents of QPACK Encoder and Decoder streams #1361

martinthomson opened this issue May 21, 2018 · 1 comment · Fixed by #1467
Labels
-qpack editorial An issue that does not affect the design of the protocol; does not require consensus.

Comments

@martinthomson
Copy link
Member

I think that the QPACK Encoder Stream comprises a sequence of these "blocks", but the draft never says that explicitly.

I can only guess at the contents of a QPACK Decoder Stream. My best guess is that it also uses "blocks". The draft needs to say what it is.
@martinthomson martinthomson added design An issue that affects the design of the protocol; resolution requires consensus. -qpack labels May 21, 2018
@afrind afrind added editorial An issue that does not affect the design of the protocol; does not require consensus. and removed design An issue that affects the design of the protocol; resolution requires consensus. labels May 22, 2018
@afrind
Copy link
Contributor

afrind commented May 22, 2018

Yes I intended it to just be a stream of the instructions defined on the decoder stream.

afrind added a commit that referenced this issue Jun 21, 2018
@afrind
Define the contents of QPACK Encoder and Decoder streams
4475cc4 
Fixes #1361

This removes the length prefixed instruction blocks on the encoder stream and replaces them with an unframed sequence of instructions.
@afrind afrind mentioned this issue Jun 21, 2018
Merged
afrind added a commit that referenced this issue Jun 26, 2018
@afrind
Define the contents of QPACK Encoder and Decoder streams (#1467)
ac5aadb 
Fixes #1361

This removes the length prefixed instruction blocks on the encoder stream and replaces them with an unframed sequence of instructions.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
-qpack editorial An issue that does not affect the design of the protocol; does not require consensus.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Define the contents of QPACK Encoder and Decoder streams quicwg/base-drafts
2 participants
@martinthomson @afrind