The SMTP_CONNECTION_URI validationRegex accepts trailing garbage after a valid prefix: its first alternative has no $ anchor, so e.g. smtp://host:25 trailing junk passes. Probe-verified; ...)$|^$ rejects the garbage forms. The bug is inherited: the legacy firestore-send-email/extension.yaml ships the identical unanchored regex (only the legacy test file carried an anchored copy that never shipped). Fix both the kit (kits/firestore-send-email/src/config.ts:171) and the legacy extension.yaml.
PR #3057 added a characterization test that pins the current unanchored acceptance - it is written to flip when this fix lands, so update it deliberately. Found by review on #3057 (IzaakGough). Ledger: #2974.
The
SMTP_CONNECTION_URIvalidationRegex accepts trailing garbage after a valid prefix: its first alternative has no$anchor, so e.g.smtp://host:25 trailing junkpasses. Probe-verified;...)$|^$rejects the garbage forms. The bug is inherited: the legacyfirestore-send-email/extension.yamlships the identical unanchored regex (only the legacy test file carried an anchored copy that never shipped). Fix both the kit (kits/firestore-send-email/src/config.ts:171) and the legacy extension.yaml.PR #3057 added a characterization test that pins the current unanchored acceptance - it is written to flip when this fix lands, so update it deliberately. Found by review on #3057 (IzaakGough). Ledger: #2974.