-
Notifications
You must be signed in to change notification settings - Fork 9
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
Support a versioning scheme #53
Comments
I vote no. This project aims to publish a specification as an IETF RFC, and ideally should present a single, coherent protocol. That we have had some iterations and even breaking changes over the course of its development is unfortunate, but I think the aim here is to get to that single protocol. Capturing and forcing compatibility with every legacy step - some of which we acknowledge have negative security implications such as Yes there has already been quite an investment by some parties to implement various versions of the draft, but the risk was always there as this is a draft. I'm not proposing breaking things intentionally or pulling the plug without consideration, but we need to think about what goal we are trying to achieve and whether these choices support that goal. |
While |
That's fine as long as the backward-compatibility won't hinder the adoption of this I-D: it would be reasonable to ask for a quick review from the http-wg. Note that this consideration is non necessarily related to |
I expect
HTTP Signatures supports a versioning scheme, so I can evolve the spec and be backward compatible during upgrades.
About protocol evolution, see https://tools.ietf.org/id/draft-thomson-postel-was-wrong-03.html
Instead
There's no version field
Proposal
1- Add a mandatory
v
field toSignature
, eg:2- The Signature is always added as a meta-header as the first element of the signed string
3- The
(v)
meta-header is not mentioned in theheaders
component of theSignature
header4- The value of
v
must be validated before processing theSignature
5- (obviously) if
v
and(v)
don't match, the hash of the signature won't match, thus making proper versioning mandatory.@msporny
The text was updated successfully, but these errors were encountered: