Fix email validation regex to properly support subdomains #1896
pull reallesee wants to merge 1 commits into bitcoin:master from reallesee:master changing 1 files +2 −2-
reallesee commented at 3:49 pm on July 15, 2025: noneImproves the regular expressions for email validation to correctly handle email addresses with multiple domain levels (subdomains). The previous regex only supported single-level domains like example.com, while the new pattern properly validates addresses like user@mail.example.com. This change follows RFC 5322 standards for email addresses and maintains backward compatibility with existing valid addresses
-
Fix email validation regex to properly support subdomains fa640c4bc1
-
murchandamus commented at 10:34 pm on July 15, 2025: contributor
Thanks. I tested the existing script and found that it already accepts email addresses with one or multiple subdomains. I also found that there are already several email addresses that follow the described pattern in the repository (e.g., karljohan-alm@garage.co.jp, rusty@rustcorp.com.au, and aj@erisian.com.au).
Therefore, the motivation for this pull request is unsubstantiated and the change unnecessary.
-
murchandamus closed this on Jul 15, 2025
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: 2025-07-29 16: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: 2025-07-29 16: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