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.

Sign up for GitHub

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

Jump to bottom

The definition of "cryptographic suite" should include or point to its whole definition #207

Closed
jyasskin opened this issue Sep 26, 2023 · 1 comment
Closed

The definition of "cryptographic suite" should include or point to its whole definition #207

jyasskin opened this issue Sep 26, 2023 · 1 comment
Assignees
@msporny
Labels
CR1 This item was processed during the first Candidate Recommendation phase. editorial This issue or PR constitutes an editorial change.

Comments

@jyasskin
Copy link
Member

https://www.w3.org/TR/vc-data-integrity/#dfn-cryptosuite doesn't say what algorithms a cryptosuite needs to provide. It should point to https://www.w3.org/TR/vc-data-integrity/#cryptographic-suites as the rest of the definition, or the actual <dfn> tag should be in the section that lists the required algorithms. Otherwise it's hard to read uses of the term and know what to expect.
@msporny msporny added the CR1 This item was processed during the first Candidate Recommendation phase. label Oct 17, 2023
@msporny
Copy link
Member

msporny commented Feb 24, 2024

Fixed in e39d4d5, closing.

@msporny msporny self-assigned this Feb 24, 2024
@msporny msporny added the editorial This issue or PR constitutes an editorial change. label Feb 24, 2024
@msporny msporny closed this as completed Feb 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@msporny msporny

Labels
CR1 This item was processed during the first Candidate Recommendation phase. editorial This issue or PR constitutes an editorial change.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jyasskin @msporny