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

Content-MD5 #93

Closed
mnot opened this issue Jun 18, 2018 · 13 comments
Closed

Content-MD5 #93

mnot opened this issue Jun 18, 2018 · 13 comments

Comments

@mnot
Copy link
Member

mnot commented Jun 18, 2018

The deprecation of Content-MD5 in 7231 was a bit too subtle; it's only mentioned in a change note, and the registry wasn't updated.

@mnot mnot added the semantics label Jun 18, 2018
@reschke
Copy link
Contributor

reschke commented Jun 19, 2018

How exactly should we update the registry?

@mnot
Copy link
Member Author

mnot commented Jun 21, 2018

change its status there to obsoleted, as it has for other selected headers.

@reschke
Copy link
Contributor

reschke commented Jun 21, 2018

History: https://trac.ietf.org/trac/httpbis/ticket/178

We removed the header field, which is different from deprecation (which we did for a new status codes). The reason might be that we hoped that a replacement for Content-MD5 would surface, and that spec would then obsolete C-MD5.

Can't we just instruct IANA to update the registry? It would be weird to talk about Content-MD5 in the new spec, when the previous one didn't have it (except in the "changes" section).

@mnot
Copy link
Member Author

mnot commented Jun 21, 2018

We can try, or we can put it in IANA considerations only.

@mnot mnot self-assigned this Oct 10, 2018
@mnot
Copy link
Member Author

mnot commented Oct 10, 2018

Proposal:

Add to the table in 4.1:

| Content-MD5 | http | obsoleted | RFC7231, Appendix B |

@reschke
Copy link
Contributor

reschke commented Oct 10, 2018

My proposal is to leave the spec alone and just tell IANA.

@royfielding
Copy link
Member

+1 to just telling IANA.

@mnot
Copy link
Member Author

mnot commented Oct 10, 2018

OK, will bring it up on the mailing list and see how that goes.

@mnot
Copy link
Member Author

mnot commented Oct 15, 2018

Brought up on the ietf-message-headers list: https://mailarchive.ietf.org/arch/msg/ietf-message-headers/ngv6EIWfWjJCIR3ilxDn8OEH38w

@aamelnikov
Copy link

IANA/IESG prefers for the action to be recorded somewhere. So I think mentioning this in a future RFC would be the best.
If people think that this is urgent, I can ask IESG approve obsoletion.

@mcmanus
Copy link

mcmanus commented Nov 14, 2018 via email

mnot added a commit that referenced this issue Feb 26, 2019
@mnot
Copy link
Member Author

mnot commented Feb 27, 2019

As per discussion in #188, should this be deprecated, not obsoleted?

@royfielding
Copy link
Member

As Julian said, if we still define the header field it is deprecated; if we don't define the header field (meaning its only definition is in obsolete documents) then it is obsoleted.

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