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-
Crypt-iQ commented at 3:56 pm on July 26, 2024: contributorThe aad shouldn’t include the garbage terminator. This tripped me up during implementation even though the text says it shouldn’t be included.
-
BIP 324: fix python aad in complete_handshake 0963e43860
-
jonatack commented at 4:07 pm on July 26, 2024: member
Initial ACK, appears to correspond to
-
jonatack added the label Proposed BIP modification on Jul 26, 2024
-
jonatack added the label Pending acceptance on Jul 26, 2024
-
stratospher commented at 4:04 am on July 27, 2024: contributorACK 0963e43. we only authenticate garbage (and not garbage terminator) as mentioned in the BIP text.
-
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.
-
jonatack merged this on Jul 27, 2024
-
jonatack closed this on Jul 27, 2024
-
jonatack removed the label Pending acceptance on Jul 27, 2024
-
jonatack removed the label Proposed BIP modification on Jul 27, 2024
-
jonatack added the label Bug fix on Jul 27, 2024
-
Crypt-iQ deleted the branch on Jul 27, 2024
Labels
Bug fix
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 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
More mirrored repositories can be found on mirror.b10c.me