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

Éric Vyncke's HTTP/3 Comment 1 #4751

Closed
LPardue opened this issue Jan 19, 2021 · 1 comment · Fixed by #4762
Closed

Éric Vyncke's HTTP/3 Comment 1 #4751

LPardue opened this issue Jan 19, 2021 · 1 comment · Fixed by #4762
Labels
-http iesg An issue raised during IESG review.
Projects
Late Stage Processing
Milestone
http-iesg

Comments

@LPardue
Copy link
Member

LPardue commented Jan 19, 2021

@evyncke said

-- Section 1.1 --
This section mixes "HTTP/1.1" and "HTTP/1.x" and it is unclear to me what the
link is between the first 2 sentences.
@LPardue LPardue added -http iesg An issue raised during IESG review. labels Jan 19, 2021
@LPardue LPardue added this to the http-iesg milestone Jan 19, 2021
@LPardue LPardue added this to Triage in Late Stage Processing via automation Jan 19, 2021
@MikeBishop
Copy link
Contributor

Two separate questions. I'm fine generalizing it to HTTP/1.x.

The link between the first two sentences is the fact that HTTP/1.x uses human-readable whitespace-delimited fields, but those attributes make parsing inconsistent; subsequent versions of HTTP/2 went a different direction.

If you're talking about the second and third, that could easily be a paragraph split; it's a change of idea under the umbrella of major changes of direction from H1 to H2+.

MikeBishop added a commit that referenced this issue Jan 19, 2021
@MikeBishop
Fixes #4751
9f98f74
@MikeBishop MikeBishop linked a pull request Jan 22, 2021 that will close this issue
Éric Vyncke's HTTP/3 Comments #4762
Merged
Late Stage Processing automation moved this from Triage to Issue Handled Jan 22, 2021
MikeBishop added a commit that referenced this issue Jan 22, 2021
@MikeBishop @martinthomson
Éric Vyncke's HTTP/3 Comments (#4762)
ce047c4 
* Fixes #4751

* Import implicit closure from -transport; fixes #4755

* Split out subordinate phrases; fixes #4758

* Reverse 1.x/1.1 polarity

* Ref transport

Co-authored-by: Martin Thomson <mt@lowentropy.net>

* TLS isn't a transport

Co-authored-by: Martin Thomson <mt@lowentropy.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
-http iesg An issue raised during IESG review.
Projects
Late Stage Processing
  
Issue Handled
Milestone
http-iesg
Development

Successfully merging a pull request may close this issue.

Éric Vyncke's HTTP/3 Comments quicwg/base-drafts
2 participants
@MikeBishop @LPardue