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

AssertionError: #9057

Closed
TARS-bot opened this issue May 18, 2024 · 3 comments
Closed

AssertionError: #9057

TARS-bot opened this issue May 18, 2024 · 3 comments

Comments

@TARS-bot
Copy link

Crash Report

This crash report was reported through the automatic crash reporting system 🤖

Traceback

Traceback (most recent call last):
  File "electrum\gui\qt\send_tab.py", line 248, in spend_max
AssertionError

AssertionError: 

Reporter

This issue was reported by 1 user(s):

Electrum Version Python Version Operating System Wallet Type Locale Stack
4.5.3 3.10.11 (tags/v3.10.11:7d4cc5a, Apr 5 2023, 00:20:04) [MSC v.1929 32 bit (Intel)] Windows-10-10.0.22631-SP0 standard pt_BR

Additional Information

The reporting user(s) did not provide additional information.

@accumulator
Copy link
Member

This happens when a BIP21 contains an explicit amount=0.

Not sure how to handle

  1. consider as invalid BIP21 uri
  2. consider as 'amountless' invoice

@SomberNight
Copy link
Member

Or, option (3): treat it as wanting to create a 0-value output. That would make sense for e.g. OP_RETURN, though I guess bip21 cannot be used for scripts that don't have a corresponding address.

  1. consider as 'amountless' invoice

There is an explicit amount, so I don't think this is correct.

I would prefer (1) or (3).

@accumulator
Copy link
Member

(1) implemented in bb4ee2b

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

3 participants