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

eth/fetcher: allow underpriced transactions in after timeout #28097

Merged

Conversation

MariusVanDerWijden
Copy link
Member

This PR will allow a previously underpriced transaction back in after a timeout of 5 minutes.
This will block most transaction spam but allow for transactions to be re-broadcasted on networks with less transaction flow.

closes #28018

@MariusVanDerWijden MariusVanDerWijden changed the title Periodically clear underpriced eth/fetcher: allow underpriced transactions in after timeout Sep 12, 2023
@MariusVanDerWijden
Copy link
Member Author

It took some time to choose the right data structure, but it should be okay now @rjl493456442

@fjl fjl merged commit 2b7bc2c into ethereum:master Sep 26, 2023
1 of 2 checks passed
@fjl fjl added this to the 1.13.2 milestone Sep 26, 2023
tyler-smith pushed a commit to blocknative/go-ethereum that referenced this pull request Oct 9, 2023
…m#28097)

This PR will allow a previously underpriced transaction back in after a timeout
of 5 minutes. This will block most transaction spam but allow for transactions to
be re-broadcasted on networks with less transaction flow.

---------

Co-authored-by: Felix Lange <fjl@twurst.com>
tyler-smith pushed a commit to blocknative/go-ethereum that referenced this pull request Oct 12, 2023
…m#28097)

This PR will allow a previously underpriced transaction back in after a timeout
of 5 minutes. This will block most transaction spam but allow for transactions to
be re-broadcasted on networks with less transaction flow.

---------

Co-authored-by: Felix Lange <fjl@twurst.com>
tyler-smith pushed a commit to blocknative/go-ethereum that referenced this pull request Oct 12, 2023
…m#28097)

This PR will allow a previously underpriced transaction back in after a timeout
of 5 minutes. This will block most transaction spam but allow for transactions to
be re-broadcasted on networks with less transaction flow.

---------

Co-authored-by: Felix Lange <fjl@twurst.com>
siosw pushed a commit to SpecularL2/go-ethereum that referenced this pull request Oct 16, 2023
…m#28097)

This PR will allow a previously underpriced transaction back in after a timeout
of 5 minutes. This will block most transaction spam but allow for transactions to
be re-broadcasted on networks with less transaction flow.

---------

Co-authored-by: Felix Lange <fjl@twurst.com>
0x366 pushed a commit to Dexcelerate/go-ethereum that referenced this pull request Nov 8, 2023
…m#28097)

This PR will allow a previously underpriced transaction back in after a timeout
of 5 minutes. This will block most transaction spam but allow for transactions to
be re-broadcasted on networks with less transaction flow.

---------

Co-authored-by: Felix Lange <fjl@twurst.com>
devopsbo3 pushed a commit to HorizenOfficial/go-ethereum that referenced this pull request Nov 10, 2023
…m#28097)

This PR will allow a previously underpriced transaction back in after a timeout
of 5 minutes. This will block most transaction spam but allow for transactions to
be re-broadcasted on networks with less transaction flow.

---------

Co-authored-by: Felix Lange <fjl@twurst.com>
devopsbo3 added a commit to HorizenOfficial/go-ethereum that referenced this pull request Nov 10, 2023
devopsbo3 added a commit to HorizenOfficial/go-ethereum that referenced this pull request Nov 10, 2023
devopsbo3 added a commit to HorizenOfficial/go-ethereum that referenced this pull request Nov 10, 2023
maoueh pushed a commit to streamingfast/go-ethereum that referenced this pull request Jan 31, 2024
…m#28097)

This PR will allow a previously underpriced transaction back in after a timeout
of 5 minutes. This will block most transaction spam but allow for transactions to
be re-broadcasted on networks with less transaction flow.

---------

Co-authored-by: Felix Lange <fjl@twurst.com>
# Conflicts:
#	eth/fetcher/tx_fetcher.go
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

Successfully merging this pull request may close these issues.

Some transactions stuck in pool with enough gas price (in a private network)
3 participants