Skip to content
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

Multitenant check endorser_info before saving #2395

Merged
merged 5 commits into from
Aug 3, 2023

Conversation

usingtechnology
Copy link
Contributor

While testing PR #2339 I was using an instance without an endorser and encountered an error while saving the new write ledger: UnboundLocalError: local variable 'endorser_alias' referenced before assignment.

This checks to see if we have endorser information before we try to persist it.

@shaangill025
Copy link
Contributor

Can you also include this change in aries_cloudagent/ledger/multiple_ledger/indy_manager.py?

@usingtechnology
Copy link
Contributor Author

yes, i will. i should have checked that... thanks!

Signed-off-by: Jason Sherman <[email protected]>
shaangill025
shaangill025 previously approved these changes Aug 2, 2023
@dbluhm dbluhm added this to the Version 0.10.0 milestone Aug 2, 2023
Signed-off-by: Jason Sherman <[email protected]>
@usingtechnology usingtechnology self-assigned this Aug 3, 2023
Signed-off-by: Jason Sherman <[email protected]>
@sonarqubecloud
Copy link

sonarqubecloud bot commented Aug 3, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@swcurran swcurran merged commit dcb858c into openwallet-foundation:main Aug 3, 2023
@usingtechnology usingtechnology deleted the fix-2339 branch August 3, 2023 18:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants