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.

Sign up for GitHub

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

Jump to bottom

Implement failing tx blacklist locally #331

Open
xlc opened this issue Mar 31, 2022 · 1 comment
Open

Implement failing tx blacklist locally #331

xlc opened this issue Mar 31, 2022 · 1 comment
Assignees
@zjb0807
Labels
feature new feature Lv. 2 relatively clear issue, might need some small digging

Comments

@xlc
Copy link
Member

xlc commented Mar 31, 2022

Substrate node by default ban failed transaction for 5 mins and attempt to resubmit failed tx will result the transaction is banned error.

eth rpc adapter should have a local in memory cache of failed tx hash and error and before submit tx, should check with the cache and return the error instead submitting it to the Substrate node.

The entry in cache should expire after 5 mins.
@gluneau gluneau added the C6-evm label Apr 29, 2022
@gluneau
Copy link
Contributor

gluneau commented Apr 29, 2022

image

@shunjizhan shunjizhan added feature new feature Lv. 2 relatively clear issue, might need some small digging Lv. 3 need some research to find out root cause or/and solution and removed C6-evm Lv. 2 relatively clear issue, might need some small digging Lv. 3 need some research to find out root cause or/and solution labels Aug 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@zjb0807 zjb0807

Labels
feature new feature Lv. 2 relatively clear issue, might need some small digging
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@xlc @gluneau @zjb0807 @shunjizhan