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

All non-malformed operations and endorsements propagate #2904

Closed
damip opened this issue Aug 24, 2022 · 5 comments
Closed

All non-malformed operations and endorsements propagate #2904

damip opened this issue Aug 24, 2022 · 5 comments
Assignees
Milestone

Comments

@damip
Copy link
Member

damip commented Aug 24, 2022

Currently, all operations and endorsements that properly deserialize will propagate to the whole network.
This presents a high risk of flooding and some checks need to be performed before propagating them.

TOOD: define which checks:

  • for ops:
    • validity period
    • available balance to pay the fees (also taking pool into account ?) ?
  • for endorsements:
    • check pos draws
    • make sure we know the targeted block and that it is not stale
    • ignore multiple endorsements per (slot, index) per address
@damip damip added this to the TEST.15.X milestone Aug 24, 2022
@damip damip changed the title All non-malformed operations propagate All non-malformed operations and endorsements propagate Aug 24, 2022
@sydhds sydhds self-assigned this Oct 28, 2022
@sydhds
Copy link
Contributor

sydhds commented Nov 3, 2022

From what I read in the code, there is already some filtering for operations:

// Propagate operations when their expire period isn't `max_operations_propagation_time` old.

and for endorsements:

// Propagate endorsements when the slot of the block they endorse isn't `max_endorsements_propagation_time` old.

I've added a quick fix here:

#3202

@sydhds
Copy link
Contributor

sydhds commented Nov 3, 2022

@damip can we discuss about what we should do next for this issue?

@damip
Copy link
Member Author

damip commented Nov 3, 2022

Oh ok, then it should be OK for now ! we can merge the correction

@sydhds
Copy link
Contributor

sydhds commented Nov 4, 2022

@damip Should we close the issue or keep it like that for now?

@AurelienFT
Copy link
Contributor

From the list above only : for endorsements: check pos draws is done in protocol

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

No branches or pull requests

3 participants