Skip to content

Conversation

@jonatack
Copy link
Member

@jonatack jonatack commented Dec 7, 2020

BIP339 currently states:

The wtxidrelay message MUST be sent in response to a version message from a peer whose protocol version is >= 70016 and prior to sending a verack. A wtxidrelay message received after a verack message MUST be ignored or treated as invalid.

as BIP339 currently states:

"The wtxidrelay message MUST be sent in response to a version
message from a peer whose protocol version is >= 70016 and
prior to sending a verack. A wtxidrelay message received after
a verack message MUST be ignored or treated as invalid."
@maflcko maflcko added the Docs label Dec 7, 2020
@maflcko
Copy link
Member

maflcko commented Dec 7, 2020

ACK 4b7b58b

Though, let's wait on merging this to make sure this doesn't cause any conflicts with pending backports

@maflcko maflcko changed the title p2p: update wtxidrelay documentation per BIP339 doc: update wtxidrelay documentation per BIP339 Dec 7, 2020
@practicalswift
Copy link
Contributor

ACK 4b7b58b

@DrahtBot
Copy link
Contributor

DrahtBot commented Dec 8, 2020

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Conflicts

Reviewers, this pull request conflicts with the following ones:

If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

@DrahtBot
Copy link
Contributor

DrahtBot commented Dec 9, 2020

🕵️ @sipa has been requested to review this pull request as specified in the REVIEWERS file.

Copy link
Contributor

@RiccardoMasutti RiccardoMasutti left a comment

Choose a reason for hiding this comment

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

ACK 4b7b58b

Copy link
Member

@luke-jr luke-jr left a comment

Choose a reason for hiding this comment

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

ACK

@maflcko maflcko merged commit 096bd37 into bitcoin:master Dec 14, 2020
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Dec 14, 2020
4b7b58b Update net_processing WTXID documentation per BIP339 (Jon Atack)

Pull request description:

  BIP339 currently states:

  *The wtxidrelay message MUST be sent in response to a version message from a peer whose protocol version is >= 70016 and prior to sending a verack. A wtxidrelay message received after a verack message MUST be ignored or treated as invalid.*

ACKs for top commit:
  MarcoFalke:
    ACK 4b7b58b
  practicalswift:
    ACK 4b7b58b
  RiccardoMasutti:
    ACK 4b7b58b

Tree-SHA512: 58ca6b197618cc73c70aa5de0a2d9d89a68b4cad9d5a708278ef17a9d6854d4362bcc384b6d29696642924977204a8fc120b31e91e2d97b6072b7b0d41c9f2dc
@jonatack jonatack deleted the update-wtxid-documentation-per-BIP339 branch December 14, 2020 20:43
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Feb 15, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants