-
Notifications
You must be signed in to change notification settings - Fork 88
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
Initial support for SD-JWT VC. #595
Conversation
identity-sdjwt/src/main/java/com/android/identity/sdjwt/Disclosure.kt
Outdated
Show resolved
Hide resolved
identity-sdjwt/src/main/java/com/android/identity/sdjwt/SdJwtVerifiableCredential.kt
Outdated
Show resolved
Hide resolved
throw AttributeNotDisclosedException("attribute $attribute not included in disclosures") | ||
} | ||
|
||
val disclosure = matchingDisclosures[0] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would multiple matches also be bad? if so singleOrNull is an elegant way to handle this
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cc @balfanz
identity-sdjwt/src/main/java/com/android/identity/sdjwt/util/JwtJsonObject.kt
Outdated
Show resolved
Hide resolved
identity-sdjwt/src/main/java/com/android/identity/sdjwt/credential/SdJwtCredential.kt
Show resolved
Hide resolved
Thanks for the review, force-pushed some of the nit fixes! Will land this. |
This adds a new identity-sdjwt library which includes support for working with the core data structures in IETF SD-JWT and SD-JWT VCs. Future commits will add support for provisioning and presentation in the 'wallet' module. Test: New unit and all unit tests pass. Co-authored-by: David Zeuthen <[email protected]> Signed-off-by: David Zeuthen <[email protected]>
This adds a new identity-sdjwt library which includes support for working with the core data structures in IETF SD-JWT and SD-JWT VCs. Future commits will add support for provisioning and presentation in the 'wallet' module.
Test: New unit and all unit tests pass.