Skip to content
This repository has been archived by the owner on May 17, 2021. It is now read-only.

[WIP] Direct headers announcement (rebase of #6494) #7129 #40

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

[WIP] Direct headers announcement (rebase of #6494) #7129 #40

wants to merge 2 commits into from

Conversation

KolbyML
Copy link
Contributor

@KolbyML KolbyML commented Aug 19, 2019

This is from bitcoin#7129

This pr adds BIP 130 direct headers announcement is negotiated with peer versions

https://github.com/bitcoin/bips/blob/master/bip-0130.mediawiki

Co-Authored-By: Suhas Daftuar sdaftuar@users.noreply.github.com
Co-Authored-By: Pieter Wuille sipa@users.noreply.github.com

KolbyML and others added 2 commits August 19, 2019 13:43
This is from bitcoin#7129

Co-Authored-By: Suhas Daftuar <sdaftuar@users.noreply.github.com>
Co-Authored-By: Pieter Wuille <sipa@users.noreply.github.com>
@KolbyML
Copy link
Contributor Author

KolbyML commented Aug 19, 2019

Don't merge this till we add the consensus folder

@JSKitty
Copy link
Member

JSKitty commented Aug 19, 2019

Beautiful! Backwards compatible as well, will wait until additional changes are done before building and testing, I'm guessing this is a work-in-progress PR? If so, I'll add [WIP] to the PR so that it doesn't get acccidently merged until complete.

@JSKitty JSKitty changed the title Direct headers announcement (rebase of #6494) #7129 [WIP ]Direct headers announcement (rebase of #6494) #7129 Aug 19, 2019
@JSKitty JSKitty changed the title [WIP ]Direct headers announcement (rebase of #6494) #7129 [WIP] Direct headers announcement (rebase of #6494) #7129 Aug 19, 2019
@JSKitty JSKitty added the Enhancement This PR / Issue brings a new feature or request label Aug 19, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Enhancement This PR / Issue brings a new feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants