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

We have two things called "frame" #128

Closed
martinthomson opened this issue Jan 6, 2017 · 13 comments
Closed

We have two things called "frame" #128

martinthomson opened this issue Jan 6, 2017 · 13 comments
Labels
-http -transport editorial An issue that does not affect the design of the protocol; does not require consensus.

Comments

@martinthomson
Copy link
Member

The transport protocol has frames, and STREAM frames can contain frames for the HTTP mapping. It's confusing. We should work out how to deal with that.

@martinthomson martinthomson added editorial An issue that does not affect the design of the protocol; does not require consensus. -http -transport labels Jan 6, 2017
@martinthomson martinthomson added the needs-discussion An issue that needs more discussion before we can resolve it. label Jan 19, 2017
@MikeBishop
Copy link
Contributor

I've been trying to consistently say "a QUIC " in the HTTP draft if I refer to a non-HTTP frame. Is that editorial convention sufficient, or were you actually looking to change terms here?

@martinthomson
Copy link
Member Author

Picking another name "segment", "chunk", "lump", "whatsit", would be superior, but that's a good working solution.

@martinthomson martinthomson removed the needs-discussion An issue that needs more discussion before we can resolve it. label Apr 27, 2017
@mnot
Copy link
Member

mnot commented Apr 28, 2017

H2 already has frames; maybe the new thing should have the new name. E.g., 'QUIC segments".

YMMV.

@mnot mnot added this to Odd Jobs in HTTP Apr 28, 2017
@mcmanus
Copy link
Contributor

mcmanus commented Apr 28, 2017 via email

@ekr
Copy link
Collaborator

ekr commented Apr 28, 2017 via email

@LPardue
Copy link
Member

LPardue commented Apr 28, 2017

A colleague of mine suggested vesicle...

@ekr
Copy link
Collaborator

ekr commented Apr 28, 2017

Also:

Cell
Fragment

@janaiyengar
Copy link
Contributor

janaiyengar commented Apr 28, 2017 via email

@MikeBishop
Copy link
Contributor

MikeBishop commented Apr 28, 2017 via email

@MikeBishop
Copy link
Contributor

The above solution has proven to be sufficient for now. Though if anyone wants to make a case for QUIC vesicles, feel free to reopen.

@LPardue
Copy link
Member

LPardue commented Nov 14, 2017

As someone that has written an I-D that needs to reference both QUIC Transport and HTTP documents, I have struggled with ensuring the correct frame-type is being referenced in text.

I have so far used the convention < X > <[ QUIC | HTTP/2 ]> frame. Changing the order is not problematic. I think there would be benefit in defining a consistent style, this can help both readers and writers of specifications. I would propose this to be added at as a section somewhere in a core document.

Furthermore, I recently took the position that HTTP/2 is the wrong term when talking about HTTP/QUIC frames. I will address this, should I call them HTTP/QUIC or would the abbreviation HQ frame also be ok? I would propose that the convention is defined in the HTTP mapping document.

@MikeBishop
Copy link
Contributor

I agree with you on that position, and one of my first editorial changes to the draft was to excise all occurrences of the term "HTTP/2 over QUIC." HTTP/2 and HTTP/QUIC are not the same protocol. Just as h2 is the ALPN token and a convenient shorthand for HTTP/2, hq (or HQ) is a perfectly reasonable shorthand for HTTP/QUIC. I tend to write it out in anything official, mostly to avoid anyone thinking I'm talking about headquarters.

I'm fine adding protocol frame naming to our notational conventions. Reopening to track that suggestion.

@MikeBishop MikeBishop reopened this Nov 14, 2017
@LPardue
Copy link
Member

LPardue commented Nov 15, 2017

SGTM, thanks Mike.

@mnot mnot removed this from Odd Jobs in HTTP Mar 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
-http -transport editorial An issue that does not affect the design of the protocol; does not require consensus.
Projects
None yet
Development

No branches or pull requests

7 participants