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

fix: Presentation Without Holder Binding should end with ~ #3468

Merged
merged 1 commit into from
Jan 10, 2023

Conversation

sandrask
Copy link
Contributor

@sandrask sandrask commented Jan 10, 2023

Fix: Presentation Without Holder Binding should end with ~

Also, rename SDJWT to CombinedPresentation and JWTSerialized to SDJWT to follow spec terms.

And add parsing and serializing of combined presentation parts with holder binding.

Holder Binding functionality will be added in separate PRs.

Closes #3467

Signed-off-by: Sandra Vrtikapa [email protected]

Fix: Presentation Without Holder Binding should end with ~

Also, rename SDJWT to CombinedPresentation and JWTSerialized to SDJWT to follow spec terms.

Closes hyperledger-archives#3467

Signed-off-by: Sandra Vrtikapa <[email protected]>
@codecov
Copy link

codecov bot commented Jan 10, 2023

Codecov Report

Merging #3468 (7b87f05) into main (e6c87c5) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #3468   +/-   ##
=======================================
  Coverage   87.65%   87.66%           
=======================================
  Files         350      350           
  Lines       47483    47516   +33     
=======================================
+ Hits        41623    41653   +30     
- Misses       4348     4350    +2     
- Partials     1512     1513    +1     
Impacted Files Coverage Δ
pkg/doc/sdjwt/common/common.go 91.22% <100.00%> (+1.94%) ⬆️
pkg/doc/sdjwt/holder/holder.go 98.68% <100.00%> (+0.01%) ⬆️
pkg/doc/sdjwt/issuer/issuer.go 92.54% <100.00%> (+0.04%) ⬆️
pkg/doc/sdjwt/verifier/verifier.go 94.78% <100.00%> (ø)
.../tinkcrypto/primitive/secp256k1/subtle/encoding.go 55.93% <0.00%> (-5.09%) ⬇️

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

@fqutishat fqutishat merged commit c8f690f into hyperledger-archives:main Jan 10, 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.

Presentation Without Holder Binding should end with ~
2 participants