italia / eudi-wallet-it-docs

Italian EUDI Wallet Technical Specifications
Creative Commons Zero v1.0 Universal
55 stars 18 forks source link

Clarify selective disclosure of the evidence claim #213

Closed fmarino-ipzs closed 7 months ago

fmarino-ipzs commented 7 months ago

In the current version of the Credential data model, we have a selective disclosable evidence field. We need to clarify the sub-claims included in the evidence. We don't enable nested SDs. Then the Holder can selectively disclosure the entire evidence, and it is not allowed to selectively disclose only a subset of the claims contained in the evidence.

balanza commented 7 months ago

My opinion:

for version 0.6.0:

for next versions:

rohe commented 7 months ago

I agree with the notion that evidence's subfield should not be selectively disclosed. To that I would like to add that one has to weigh the benefit of being able to enable nested/recursive selective disclosures against the increased complexity it will bring to the implementations.

peppelinux commented 7 months ago

Over the past few weeks, my team has addressed the following:

To overcome these limitations, we've decided to:

Below I share a pull request that introduces this "feature" within the OpenID4VCI metadata (and divides AS metadata from OpenID4VCI metadata as well). A subsequent pull request will modify the PID data model.

https://github.com/italia/eudi-wallet-it-docs/pull/216/files#diff-94e69d33268a4f2df6ac286d8ab2f24606e869d55c6d8ed1bc35884c14e12abaR1076