-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Bump PROTOCOL_VERSION #3287
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
Bump PROTOCOL_VERSION #3287
Conversation
We removed "alert" p2p message and changed "mempool" message behaviour a bit. It probably makes sense to bump PROTOCOL_VERSION now. This should also help to distinguish v0.15 nodes from late v0.14.0.x ones.
|
Something to consider for https://github.com/nightlydash/dash-seeder as well? |
|
@schinzelh yep, good point 👍 (actually, it's 70215 already but it won't hurt to keep things aligned nightlydash/dash-seeder#24) |
codablock
left a comment
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.
utACK
b0c5dc4 to
aa144e4
Compare
codablock
left a comment
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.
re-utACK
- Related to dashpay/dash#3287
thephez
left a comment
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.
utACK
We removed "alert" p2p message and changed "mempool" message behaviour a bit. It probably makes sense to bump PROTOCOL_VERSION now. This should also help to distinguish v0.15 nodes from late v0.14.0.x ones.
We removed "alert" p2p message and changed "mempool" message behaviour a bit. It probably makes sense to bump PROTOCOL_VERSION now. This should also help to distinguish v0.15 nodes from late v0.14.0.x ones.
Rationale:
We no longer need to support pre-v0.14 nodes.We do have quite a few mobile SPV wallets running on low end Android phones though. Postponing MIN_PEER_PROTO_VERSION bump till later.