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

babe, grandpa: set longevity for equivocation report transactions #8076

Merged
5 commits merged into from
Feb 10, 2021

Conversation

andresilva
Copy link
Contributor

@andresilva andresilva commented Feb 8, 2021

Currently the equivocation report transactions never expire, they will only be pruned from the queue if they are included in a block. Any equivocation report is only useful for the period that the given offender is bonded. This PR makes it so that any equivocation report transaction has longevity equal to the bonding duration in blocks.

polkadot companion: paritytech/polkadot#2404

@andresilva andresilva added A0-please_review Pull request needs code review. 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 8, 2021
@smisty

This comment has been minimized.

@bkchr bkchr requested a review from gui1117 February 10, 2021 12:08
@gui1117
Copy link
Contributor

gui1117 commented Feb 10, 2021

bot merge

@ghost
Copy link

ghost commented Feb 10, 2021

Trying merge.

@ghost ghost merged commit ce018b8 into master Feb 10, 2021
@ghost ghost deleted the andre/equivocation-report-longevity branch February 10, 2021 13:51
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-please_review Pull request needs code review. 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.

5 participants