Skip to content
This repository has been archived by the owner on Mar 27, 2024. It is now read-only.

feat: Add structured claims option #3481

Merged
merged 1 commit into from
Jan 18, 2023

Conversation

sandrask
Copy link
Contributor

Added:

  • structured claims option to issuer
  • verify digests in structured claims for holder
  • structured claims processing to verifier

Closes #3478

Signed-off-by: Sandra Vrtikapa sandra.vrtikapa@securekey.com

@codecov
Copy link

codecov bot commented Jan 18, 2023

Codecov Report

Merging #3481 (72fab47) into main (6123411) will decrease coverage by 0.01%.
The diff coverage is 93.07%.

@@            Coverage Diff             @@
##             main    #3481      +/-   ##
==========================================
- Coverage   87.68%   87.67%   -0.01%     
==========================================
  Files         350      350              
  Lines       47670    47762      +92     
==========================================
+ Hits        41799    41876      +77     
- Misses       4355     4365      +10     
- Partials     1516     1521       +5     
Impacted Files Coverage Δ
pkg/doc/sdjwt/verifier/verifier.go 90.82% <62.50%> (-1.38%) ⬇️
pkg/doc/sdjwt/issuer/issuer.go 92.42% <92.30%> (-2.44%) ⬇️
pkg/doc/sdjwt/common/common.go 91.32% <96.38%> (+0.09%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@sandrask sandrask force-pushed the issue-3478 branch 2 times, most recently from 19e827d to 41c5c4b Compare January 18, 2023 17:02
@@ -756,4 +759,26 @@ func buildJWS(signer afjose.Signer, claims interface{}) (string, error) {
return jws.SerializeCompact(false)
}

func printObject(t *testing.T, name string, obj interface{}) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should mark helper func with t.Helper()

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

Added:
- structured claims option to issuer
- verify digests in structured claims for holder
- structured claims processing to verifier

Closes hyperledger-archives#3478

Signed-off-by: Sandra Vrtikapa <sandra.vrtikapa@securekey.com>
@fqutishat fqutishat merged commit 25d5d00 into hyperledger-archives:main Jan 18, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

SD-JWT: Add structured claims option
3 participants