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

fix(statedb-cmd): improve error handling in genesis_utxo.rs #2094

Merged
merged 1 commit into from
Jul 7, 2024

Conversation

popcnt1
Copy link
Collaborator

@popcnt1 popcnt1 commented Jul 7, 2024

Summary

The code for generating UTXO updates has been enhanced by introducing better and more robust error handling mechanisms. This update also improves the way it handles the generation of bitcoin and rooch addresses, simplifying the logic and reducing code redundancy. These changes are aimed at preventing unexpected crashes and providing clearer error messages if an issue occurs.

example:

UTXOData { txid: "41a3e9ee1910a2d40dd217bbc9fd3638c40d13c8fdda8a0aa9d49a2b4a199422", vout: 2, value: 5376, script: "036c6565662c206f6e7464656b2c2067656e6965742e2e2e202020202020202020", script_type: "p2pk", address: "" }

The code for generating UTXO updates has been enhanced by introducing better and more robust error handling mechanisms. This update also improves the way it handles the generation of bitcoin and rooch addresses, simplifying the logic and reducing code redundancy. These changes are aimed at preventing unexpected crashes and providing clearer error messages if an issue occurs.
@popcnt1 popcnt1 requested review from wow-sven and pause125 as code owners July 7, 2024 14:29
Copy link

vercel bot commented Jul 7, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
rooch-portal ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 7, 2024 2:29pm
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
rooch ⬜️ Ignored (Inspect) Visit Preview Jul 7, 2024 2:29pm

@popcnt1 popcnt1 merged commit ba898a8 into rooch-network:main Jul 7, 2024
5 of 7 checks passed
@popcnt1 popcnt1 deleted the fix.utxo branch July 7, 2024 14:29
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.

1 participant