decentralized-identity / decentralized-identity/claim-format-registry

Add di_vc and di_vp formats

Open
#8 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
No language data
Stars
2
Forks
4
PR merge metrics
No merged PRs in 30d

Description

With the move towards the new data integrity vc spec (https://www.w3.org/TR/vc-data-integrity), it is now recommend to use `DataIntegrityProof` proof type.

This means that the actual crypt schema that is used is hidden, and thus the filter becomes a lot less useful.
Therefore I think it might be useful to register a new `di_vc` and `di_vp` claim format which define a new `cryptosuite` property:

```json
{
"^di_vc$|^di_vp$": {
"type": "object",
"additionalProperties": false,
"properties": {
// used for proof.type values (either DataIntegrityProof or custom one)
"proof_type": {
"type": "array",
"minItems": 1,
"items": { "type": "string" }
},
// used for cryptosuite property of `DataIntegrityProof`, if `DataIntegrityProof` is include in `proof_type`)
"cryptosuite": {
"type": "array",
"minItems": 1,
"items": { "type": "string" }
}
}
}
}
```

An alternative would be to extend the ldp_vc and ldp_vp with an optional `cryptosuite` property that can be used if the `DataIntegrityProof` type is used:

```json
{
"^ldp_vc$|^ldp_vp$|^ldp$": {
"type": "object",
"additionalProperties": false,
"properties": {
"proof_type": {
"type": "array",
"minItems": 1,
"items": { "type": "string" }
},
"cryptosuite": {
"type": "array",
"minItems": 1,
"items": { "type": "string" }
}
}
}
}
```

as data integrity is also the new name of the spec, it might be good to register the new `di_vc` either way.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.