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
Reconcile 'content' and 'payload' #1703
Conversation
I did a complete pass through and it wasn't that bad. As we agreed, payload is the bytes that we protect, content is the more abstract notion of what needs to be communicated (for which we use frames). That was already mostly right, apart from a few places. Closes #1676.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for doing this.
draft-ietf-quic-transport.md
Outdated
@@ -1109,7 +1109,7 @@ Stateless Reset do not contain frames. | |||
| Frame N (*) ... | |||
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+ | |||
~~~ | |||
{: #packet-frames title="Contents of Protected Payload"} | |||
{: #packet-frames title="QUIC Payload"} | |||
|
|||
Protected payloads MUST contain at least one frame, and MAY contain multiple |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You removed the term "protected payload" from the title of the figure, but not from the following paragraph.
draft-ietf-quic-transport.md
Outdated
@@ -566,8 +566,8 @@ wishes to perform a stateless retry (see {{stateless-retry}}). | |||
|
|||
A Retry packet (shown in {{retry-format}}) only uses the invariant portion of | |||
the long packet header {{QUIC-INVARIANTS}}; that is, the fields up to and | |||
including the Destination and Source Connection ID fields. The contents of the | |||
Retry packet are not protected. Like Version Negotiation, a Retry packet | |||
including the Destination and Source Connection ID fields. The Payload of a |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could go either way, but I'd lean toward not capitalizing this usage, or saying "The Payload field"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I realized that we say that it has a Payload here, then say that it doesn't in the next sentence, so I changed this more.
3e798ec
to
f78d9f8
Compare
I did a complete pass through and it wasn't that bad. As we agreed,
payload is the bytes that we protect, content is the more abstract
notion of what needs to be communicated (for which we use frames). That
was already mostly right, apart from a few places.
Oh, and I didn't mention this, but I think that it's worth saying it. This isn't in the definitions section. We don't need to load that down with these little editorial conventions. We do that and it becomes unusable.
Closes #1676.