Skip to content
This repository has been archived by the owner on Apr 4, 2024. It is now read-only.

Apply base fee to native cosmos transaction #951

Closed
yihuang opened this issue Feb 22, 2022 · 3 comments
Closed

Apply base fee to native cosmos transaction #951

yihuang opened this issue Feb 22, 2022 · 3 comments

Comments

@yihuang
Copy link
Contributor

yihuang commented Feb 22, 2022

Proposal: Apply base fee to native cosmos transaction

Current behavior:

Currently, cosmos tx only uses min-gas-prices which are configured per validator, but they occupy the same block space as eth tx, which increases the base fee of eth tx, there's an asymmetric here, maybe exploitable for ddos attack.

Desired behavior:

Apply dynamic fee to both cosmos and eth txs.

Use case: [Why is this important (helps with prioritizing requests)]

Requests may be closed if we're not actively planning to work on them.

@github-actions
Copy link

github-actions bot commented Apr 9, 2022

This issue is stale because it has been open 45 days with no activity. Remove Status: Stale label or comment or this will be closed in 7 days.

@yihuang
Copy link
Contributor Author

yihuang commented Jun 16, 2022

ask for reopen

  • Upgrade to 0.46
  • Apply the feemarket module to the new ante handler options.

Postpone to after cosmos-sdk 0.46 rollbacked to tendermint 0.34 with backported prioritized mempool.

@yihuang
Copy link
Contributor Author

yihuang commented Jul 11, 2022

This can be supported easily after update to cosmos-sdk 0.46: #1168, shall we reopen this?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant