Skip to content

Issueasset tx with blind parameter false is not accepted after the change for more than 21M #1443

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

Open
Mixa84 opened this issue Apr 2, 2025 · 2 comments

Comments

@Mixa84
Copy link

Mixa84 commented Apr 2, 2025

Discovered this issue while merging recent RC changes on Sequentia, concretely 23.3.0 RC2 (#1338) . Our issueasset is by default unblinded and wallet_elements_21million.py was failing. While debugging the issue I've found that MoneyRange() check in confidential_validation.cpp wasn't adapted to do it only for policy asset but this check is done only in case that amount is explicit, so the transaction was rejected on broadcast.

@delta1 @psgreco

Mixa84 added a commit to SequentiaSEQ/sequentia that referenced this issue Apr 2, 2025
Mixa84 added a commit to SequentiaSEQ/sequentia that referenced this issue Apr 2, 2025
Mixa84 added a commit to SequentiaSEQ/sequentia that referenced this issue Apr 2, 2025
@delta1
Copy link
Member

delta1 commented Apr 3, 2025

Hi @Mixa84 thank you for reporting this, slightly simpler implementation against master here: #1445

@Mixa84
Copy link
Author

Mixa84 commented Apr 3, 2025

Thanks, I didn't want to import a new header and use a static Params object in confidental_validation.cpp, but yeah this is quite simpler.

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

No branches or pull requests

2 participants