BIP 434 Support: Peer feature negotiation #35221

pull ajtowns wants to merge 6 commits into bitcoin:master from ajtowns:202604-bip434-support changing 16 files +180 −17
  1. ajtowns commented at 11:48 AM on May 6, 2026: contributor

    Adds support for BIP 434.

  2. DrahtBot commented at 11:48 AM on May 6, 2026: contributor

    <!--e57a25ab6845829454e8d69fc972939a-->

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

    <!--006a51241073e994b41acfe9ec718e94-->

    Code Coverage & Benchmarks

    For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/35221.

    <!--021abf342d371248e50ceaed478a90ca-->

    Reviews

    See the guideline for information on the review process.

    Type Reviewers
    Concept ACK fjahr, sedited

    If your review is incorrectly listed, please copy-paste <code>&lt;!--meta-tag:bot-skip--&gt;</code> into the comment that the bot should ignore.

    <!--174a7506f384e20aa4161008e828411d-->

    Conflicts

    Reviewers, this pull request conflicts with the following ones:

    • #28463 (p2p: Increase inbound capacity for block-relay only connections by mzumsande)

    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.

    <!--5faf32d7da4f0f540f40219e4f7537a3-->

    LLM Linter (✨ experimental)

    Possible places where named args for integral literals may be used (e.g. func(x, /*named_arg=*/0) in C++, and func(x, named_arg=0) in Python):

    • MakeAndPushFeature(pfrom, NetMsgFeature::DUMMY, uint32_t{0}) in src/net_processing.cpp

    <sup>2026-05-06 19:58:52</sup>

  3. ajtowns force-pushed on May 6, 2026
  4. DrahtBot added the label CI failed on May 6, 2026
  5. fjahr commented at 12:23 PM on May 6, 2026: contributor

    Concept ACK

  6. sedited commented at 12:43 PM on May 6, 2026: contributor

    Concept ACK

  7. ajtowns force-pushed on May 6, 2026
  8. ajtowns force-pushed on May 6, 2026
  9. serialize: string_view serialization
    Allows `stream << sv` to serialize from a string_view directly, without
    converting to a string or similar first.
    cad323f630
  10. serialize: add LimitedVectorFormatter cda4d113d2
  11. net: Add AdvertisedVersion() for protocol version advertised to a peer 88efda2743
  12. BIP434: FEATURE message support
    BIP434 defines FEATURE messages which are sent between VERSION and VERACK
    to indicate support for new P2P protocol features. This commit provides
    the infrastructure for easily using BIP434 negotiation when implementing
    such new P2P protocol features. Note that advertised protocol version
    is bumped to 70017, as per BIP434's specification.
    f51cb08f70
  13. test_framework: BIP 434 support a1fec832e5
  14. dummy bip434 feature
    This is a dummy commit to ensure that everything compiles when it's used, will
    be dropped before PR is undrafted.
    35502ef92b
  15. DrahtBot added the label Needs rebase on May 6, 2026
  16. ajtowns force-pushed on May 6, 2026
  17. DrahtBot removed the label Needs rebase on May 6, 2026

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-05-11 12:12 UTC

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