Headers announcement for nodes that can do headers. #8961

pull rebroad wants to merge 1 commits into bitcoin:master from rebroad:AnnounceUsingHeaders changing 1 files +11 −6
  1. rebroad commented at 3:50 PM on October 18, 2016: contributor

    Announce blocks using headers instead of block invs to nodes that can process headers.

  2. Headers announcement for nodes that can do headers. cb9fdf9b43
  3. MarcoFalke added the label P2P on Oct 18, 2016
  4. MarcoFalke commented at 7:30 PM on October 18, 2016: member

    Have you tried compiling this and running the test suite locally?

  5. sdaftuar commented at 7:35 PM on October 18, 2016: member

    NACK. Please see BIP130, which was implemented in #7129.

  6. MarcoFalke closed this on Oct 18, 2016

  7. rebroad commented at 5:55 AM on October 19, 2016: contributor

    @sdaftuar Yes, this builds on top of BIP130 as BIP130 missed a few tricks. In fact, fPreferHeaders couuld have been removed with this pull, but I've kept it in as it can be useful for debugging (i.e. showing the difference between headers being sent to nodes capable, and nodes which prefer them). @MarcoFalke I will be running make check shortly as this is a good idea for sure. Why did you close this please?

  8. MarcoFalke locked this on Sep 8, 2021

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-04-22 18:15 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me