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

Charge gas for masp fee payment #2592

Closed
Tracked by #2530 ...
grarco opened this issue Feb 12, 2024 · 0 comments · Fixed by #2619
Closed
Tracked by #2530 ...

Charge gas for masp fee payment #2592

grarco opened this issue Feb 12, 2024 · 0 comments · Fixed by #2619
Assignees
Labels
gas MASP pre-mainnet Must happen before mainnet.

Comments

@grarco
Copy link
Collaborator

grarco commented Feb 12, 2024

Currently the masp transaction for fee payment does not charge gas (expect for the extra space required by the tx itself) and therefore does not pay fees. We should instead charge gas for its execution too and this should be accounted in the tx GasLimit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
gas MASP pre-mainnet Must happen before mainnet.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants