script: allow hyphens in deputy email domains #2283

pull kriss39 wants to merge 1 commits into bitcoin:master from kriss39:fix/deputies-email-domain-validation changing 1 files +1 −1
  1. kriss39 commented at 1:28 PM on September 11, 2026: contributor

    BIP 3 specifies that the Deputies header uses the same format as the Authors header, but buildtable.pl currently validates them with slightly different domain patterns.

    Authors accepts hyphens in domain names, while the generic email-field check used by Deputies and legacy Editor fields does not. As a result, a valid address such as alice@bitcoin-dev.org is accepted for an author but rejected for a deputy.

    This change allows hyphens in the generic email domain matcher as well, keeping the validation consistent with the Authors format.

  2. script: allow hyphens in deputy email domains afc982a096
  3. murchandamus approved
  4. murchandamus commented at 7:16 PM on September 11, 2026: member

    ACK afc982a0961c65995d8b5cd76a8c11ce2b6d2d65. Good catch, thanks.

  5. murchandamus added the label Bug fix on Sep 11, 2026
  6. murchandamus added the label CI on Sep 11, 2026
  7. murchandamus merged this on Sep 11, 2026
  8. murchandamus closed this on Sep 11, 2026

Labels

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: 2026-09-13 01:10 UTC

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