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

Cite Multibase normatively #102

Merged
merged 1 commit into from
Jul 5, 2023
Merged

Cite Multibase normatively #102

merged 1 commit into from
Jul 5, 2023

Conversation

jyasskin
Copy link
Member

@jyasskin jyasskin commented Jun 28, 2023

One instance was missing the citation, and the other had a MUST combined with a non-normative citation.


Preview | Diff

One instance was missing the citation, and the other had a MUST combined with a non-normative citation.
@msporny
Copy link
Member

msporny commented Jul 2, 2023

@jyasskin wrote:

One instance was missing the citation, and the other had a MUST combined with a non-normative citation.

This PR will be merged, just noting that if the IETF work doesn't progress on an appropriate timeline, we will most likely just define the values directly in the cryptosuite specifications that use the values, remove the normative references to the Multiformats specifications at IETF, and call it a day. That is, the reference will be informative in the Data Integrity specification, and the normative values will be placed directly in the cryptosuite specifications.

More detail here:

https://lists.w3.org/Archives/Public/public-vc-wg/2023Jul/0000.html

You can see how these values are defined today in the various cryptosuite specifications:

https://w3c.github.io/vc-di-eddsa/#multikey
https://w3c.github.io/vc-di-ecdsa/#multikey
https://w3c.github.io/vc-di-bbs/#multikey

@msporny
Copy link
Member

msporny commented Jul 5, 2023

Normative, multiple positive reviews, no changes requested, no objections, merging.

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

Successfully merging this pull request may close these issues.

5 participants