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

fix: Wallet query multi-descriptor fix #3544

Conversation

sandrask
Copy link
Contributor

@sandrask sandrask commented Mar 3, 2023

If we have two input descriptors with two credentials and one descriptor is satisfied with two credentials then we have failure depending on the order of processing.

Closes #3543

If we have two input descriptors with two credentials and one descriptor is satisfied with two credentials then we have failure depending on the order of processing.

Closes hyperledger-archives#3543

Signed-off-by: Sandra Vrtikapa <sandra.vrtikapa@securekey.com>
@sandrask sandrask force-pushed the wallet-multi-descriptors-fix branch from e4c7b3b to e43e9f6 Compare March 3, 2023 19:42
@codecov
Copy link

codecov bot commented Mar 3, 2023

Codecov Report

Merging #3544 (e43e9f6) into main (5cac1cf) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #3544   +/-   ##
=======================================
  Coverage   87.73%   87.73%           
=======================================
  Files         351      351           
  Lines       48428    48428           
=======================================
  Hits        42486    42486           
  Misses       4402     4402           
  Partials     1540     1540           
Impacted Files Coverage Δ
pkg/internal/ldtestutil/document_loader.go 82.14% <ø> (ø)
pkg/doc/presexch/definition.go 85.85% <100.00%> (ø)

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

@sudeshrshetty sudeshrshetty merged commit 46985c2 into hyperledger-archives:main Mar 6, 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.

Wallet query multi-descriptor fix
4 participants