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

Allow mutliple VCs in VP and validate VP without VC #46

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

sake
Copy link

@sake sake commented Mar 18, 2024

The patches contain two changes.

  1. A VP may contain multiple VCs, so the VerifiablePresentation class is modified to retrieve a list of VCs. A possible use multiple VCs is that a self signed VC containing user asserted data might be presented together with a VC asserted by a third party.
  2. VPs might contain no VC. The Validation class enforces the presence of a VC, however OID4VCI uses ldp proofs to claim key ownership. Such a VP contains no VC. In order to retain backwards compatibility, a boolean flag in has been introduced in the validate method.

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.

None yet

1 participant