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

Rename "digest-headers" -> "digest-fields" #1632

Closed
ioggstream opened this issue Sep 1, 2021 · 5 comments
Closed

Rename "digest-headers" -> "digest-fields" #1632

ioggstream opened this issue Sep 1, 2021 · 5 comments

Comments

@ioggstream
Copy link
Contributor

ioggstream commented Sep 1, 2021

Discussion

The "Digest Headers" document could be renamed to "Digest Fields" since they can be used in trailer sections.

The new name could take into account the fact that we're going to have two fields too:

  • Digest
  • Content-Digest

WDYT? @LPardue @mnot

Note

Given how similar Want-Digest and Want-Content-Digest are, and that they share the want-digest-value we can probably editorialize this a bit. But in order not to block this PR, we can do that work in a follow up. #1543 (comment)

@LPardue
Copy link
Contributor

LPardue commented Sep 10, 2021

Renaming the title of the doc is oberwhelmingly good in my mind, I hope the chairs agree

@LPardue
Copy link
Contributor

LPardue commented Sep 10, 2021

The point I made about Want-* headers is tangential to the title, I'll follow up with a PR

@jricher
Copy link
Contributor

jricher commented Sep 10, 2021

I think this is a good idea.

@MikeBishop
Copy link
Contributor

Title is reasonable; probably not worth changing the filename.

@mnot
Copy link
Member

mnot commented Sep 13, 2021

Sure, this seems obvious and good.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

5 participants