-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
feat(txpool) - add flag for local tx propagation #3977
Conversation
Codecov Report
... and 9 files with indirect coverage changes
Flags with carried forward coverage won't be shown. Click here to find out more.
|
e5e79a3
to
2ac06c3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice
this is functionally complete, only have a few nits
@@ -185,6 +188,11 @@ impl EthTransactionValidatorBuilder { | |||
self.eip1559 = eip1559; | |||
self | |||
} | |||
/// Sets the local transaction propagation toggle |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This needs a bit more context:
basically if set to true (default), all locally received transactions (via RPC for example) will be propagated over the network
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
makes sense, added a bit more context
2ac06c3
to
98a49f4
Compare
98a49f4
to
75313b4
Compare
54f4a15
to
143de80
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice, tysm
should close #3966