forked from bitcoin/bitcoin
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
feat: implement Platform Ban PoSe DIP-0031 #6613
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
Open
knst
wants to merge
22
commits into
dashpay:develop
Choose a base branch
from
knst:platform-pose-ban
base: develop
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 1 commit
Commits
Show all changes
22 commits
Select commit
Hold shift + click to select a range
521a8e1
test: sort p2p messages list in p2p.py
knst 775114e
test: removed on_feefilter that has never been implemented for Dash
knst fbf89d6
feat!: add new protocol version PLATFORM_BAN_VERSION = 70236
knst 2cd44f0
feat!: implementation of Platform PoSe Ban p2p message
knst 361991f
test: add platforban message to test framework
knst 2dd8026
test: functional test p2p_platform_ban.py
knst f316bf9
fix: add missing LOCK for m_platform_ban
knst 9b6597e
fix: add list of valid mns for mine_quorum helper in p2p_platform_ban.py
knst 00e047d
fix: bump version in masternode meta store
knst 24797be
refactor: use std::move to make a copy of PlatformBan message
knst 7d39124
refactor: use LRU cache instead map to limit size of seen platform ba…
knst c739f3a
refactor: fixes based on code review and better documenting of classes
knst e73e620
refactor: rename argument name from from to node
knst 497a04c
refactor: rename argument name from from to node II
knst 853e9c8
refactor: rename day_of_blocks to PLATFORM_BAN_WINDOW_BLOCKS
knst 6e39350
fix: add missing call of EraseObjectRequest for platform ban p2p message
knst 71825d1
refactor: let PostProcessMessage do the job
knst e0d6327
refactor: let PostProcessMessage do the job II
knst e8b01f4
fix: return ret instead {}, missing usage
knst 4221644
fix: misusing m_inventory for ProcessMessageRet
knst c687022
fix: enforce quorum hash while validation platform ban p2p message
knst c191c53
fix: no flipping ban/unban if it's the same height for p2p message fo…
knst File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
but without
=
it will be flipping for the same height...