search

w3c / vc-data-integrity

W3C Data Integrity Specification
https://w3c.github.io/vc-data-integrity/
Other
42 stars 19 forks source link

cryptosuite missing from property list in 2.1 Proofs #232

Closed HelgeKrueger closed 3 months ago

HelgeKrueger commented 9 months ago

See https://www.w3.org/TR/vc-data-integrity/#proofs

My understanding is that cryptosuite is mandatory for the proof elements.

dlongley commented 9 months ago

Yes, and we need to mention that verificationMethod can be optional like the other properties if the cryptosuite provides another mechanism by which to obtain it. In short, it's really type and cryptosuite that are hard requirements to enable extensibility.

msporny commented 4 months ago

In short, it's really type and cryptosuite that are hard requirements to enable extensibility.

Hmm, no, cryptosuite is only required on DataIntegrityProofs... there are other types of proofs (like all the legacy ones) that don't require it. We'd break backwards compat if we required it for all proof objects.

msporny commented 3 months ago

PR #266 was raised, discussed, and merged to address this issue. Closing.