-
Notifications
You must be signed in to change notification settings - Fork 516
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
Fixes and cleanups for issue-credential 1.0 #1619
Fixes and cleanups for issue-credential 1.0 #1619
Conversation
andrewwhitehead
commented
Feb 5, 2022
- Improve logging
- Use transactions for better state management under Askar
- Add more checks around the credential exchange state in case of repeated messages
Codecov Report
@@ Coverage Diff @@
## main #1619 +/- ##
==========================================
- Coverage 95.71% 95.67% -0.05%
==========================================
Files 528 528
Lines 32590 32657 +67
==========================================
+ Hits 31194 31243 +49
- Misses 1396 1414 +18 |
126b3bd
to
6642ff4
Compare
6642ff4
to
41fb940
Compare
Signed-off-by: Andrew Whitehead <[email protected]>
Signed-off-by: Andrew Whitehead <[email protected]>
Signed-off-by: Andrew Whitehead <[email protected]>
Signed-off-by: Andrew Whitehead <[email protected]>
Signed-off-by: Andrew Whitehead <[email protected]>
…dd for_update option to retrieval methods Signed-off-by: Andrew Whitehead <[email protected]>
Signed-off-by: Andrew Whitehead <[email protected]>
41fb940
to
d7f248f
Compare
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.
Confirm that this improves the stability of issuing credentials (tested using the performance demo). I still get some errors but not as frequent.
(had a successful run with |