Skip to content
This repository was archived by the owner on Nov 15, 2023. It is now read-only.

Use only fresh DisputeStatement-s for dispute spam detection (#4921)#4957

Merged
rphmeier merged 1 commit intoparitytech:masterfrom
tdimitrov:fresh-nodes-spam-slots
Feb 22, 2022
Merged

Use only fresh DisputeStatement-s for dispute spam detection (#4921)#4957
rphmeier merged 1 commit intoparitytech:masterfrom
tdimitrov:fresh-nodes-spam-slots

Conversation

@tdimitrov
Copy link
Copy Markdown
Contributor

@tdimitrov tdimitrov commented Feb 21, 2022

handle_import_statements() from Dispute-Coordinator does spam detection
by counting unconfirmed dispute statements from each validator.

This patch adjusts the logic so that only fresh dispute statements are
counted.

Fresh dispute statement in this case means a statement which was not
loaded from overlay db.

Fixes: #4921

@tdimitrov tdimitrov added B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. labels Feb 21, 2022
Copy link
Copy Markdown
Member

@eskimor eskimor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@tdimitrov tdimitrov force-pushed the fresh-nodes-spam-slots branch from 761611e to 9b9746e Compare February 22, 2022 13:23
Copy link
Copy Markdown
Contributor

@drahnr drahnr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, a few small nits

…ech#4921)

handle_import_statements() from  Dispute-Coordinator does spam detection
by counting unconfirmed dispute statements from each validator.

This patch adjusts the logic so that only fresh dispute statements are
counted.

Fresh dispute statement in this case means a statement which was not
loaded from overlay db.
@tdimitrov tdimitrov force-pushed the fresh-nodes-spam-slots branch from 9b9746e to 4703d6b Compare February 22, 2022 15:44
@rphmeier rphmeier merged commit 8145a0f into paritytech:master Feb 22, 2022
@tdimitrov tdimitrov deleted the fresh-nodes-spam-slots branch February 23, 2022 12:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Only consider fresh votes in node side dispute spam slots

4 participants