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

Add an extra check for absurdly high fee before spending time creating multiple joinsplits #2981

Open
bitcartel opened this issue Feb 23, 2018 · 0 comments
Assignees
Labels
A-wallet Area: Wallet F-tx-fees Feature: Transaction fees

Comments

@bitcartel
Copy link
Contributor

Per comment here:

#2797 (comment)

There is already a check based on the amount being spent.

Add another check that a user supplied fee is not absurd, based on estimated tx size, as used by the mempool, before creating a transaction.

@bitcartel bitcartel added the A-wallet Area: Wallet label Feb 23, 2018
@bitcartel bitcartel self-assigned this Feb 23, 2018
@daira daira added the F-tx-fees Feature: Transaction fees label Mar 23, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-wallet Area: Wallet F-tx-fees Feature: Transaction fees
Projects
No open projects
Continuous Improvement
  
Awaiting triage
Development

No branches or pull requests

2 participants