BIP 324: fix python aad in complete_handshake #1655

pull Crypt-iQ wants to merge 1 commits into bitcoin:master from Crypt-iQ:bip324_aad changing 1 files +1 −1
  1. Crypt-iQ commented at 3:56 pm on July 26, 2024: contributor
    The aad shouldn’t include the garbage terminator. This tripped me up during implementation even though the text says it shouldn’t be included.
  2. BIP 324: fix python aad in complete_handshake 0963e43860
  3. jonatack added the label Proposed BIP modification on Jul 26, 2024
  4. jonatack added the label Pending acceptance on Jul 26, 2024
  5. stratospher commented at 4:04 am on July 27, 2024: contributor
    ACK 0963e43. we only authenticate garbage (and not garbage terminator) as mentioned in the BIP text.
  6. jonatack commented at 2:01 pm on July 27, 2024: member

    ACK 0963e43860e2c8e41c0821e2bc442bdd337a0ded

    As implemented in Bitcoin Core and in its functional tests, and per line 360 of the BIP:

    Both the initiator and the responder set the AAD of the first encrypted packet they send after the garbage terminator (i.e., either an optional decoy packet or the version packet) to the garbage they have just sent, not including the garbage terminator.

  7. jonatack merged this on Jul 27, 2024
  8. jonatack closed this on Jul 27, 2024

  9. jonatack removed the label Pending acceptance on Jul 27, 2024
  10. jonatack removed the label Proposed BIP modification on Jul 27, 2024
  11. jonatack added the label Bug fix on Jul 27, 2024
  12. Crypt-iQ deleted the branch on Jul 27, 2024

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bips. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-12-09 07:10 UTC

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