buildtable.pl: tighten date validation (disallow 00 month/day) #1988

pull viktorking7 wants to merge 2 commits into bitcoin:master from viktorking7:patch-1 changing 1 files +2 −1

viktorking7 commented at 8:44 AM on September 29, 2025: none
Description:
- Enforces valid month (01–12) and day (01–31) in the preamble date regex.
- Prevents invalid inputs like 2023-00-00 from passing validation.
Update buildtable.pl 79be58b2f6
katesalazar commented at 10:35 AM on September 29, 2025: contributor

https://www.youtube.com/watch?v=4yAQcKhv_5k
murchandamus added the label Typo-to-be-collected on Sep 29, 2025

in scripts/buildtable.pl:202 in 79be58b2f6

 198 | @@ -199,7 +199,8 @@
 199 |  				die "First Comments-URI must be exactly \"$first_comments_uri\" in $fn" unless $val eq $first_comments_uri;
 200 |  			}
 201 |  		} elsif (exists $DateField{$field}) {
 202 | -			die "Invalid date format in $fn" unless $val =~ /^20\d{2}\-(?:0\d|1[012])\-(?:[012]\d|30|31)$/;
 203 | +			# Enforce valid month (01-12) and day (01-31), disallow 00

jonatack commented at 4:10 PM on October 3, 2025:

			# Enforce date format 20XX-MM-DD, where XX is 00-99, MM is 01-12 and DD is 01-31.

jonatack commented at 4:11 PM on October 3, 2025: member

ACK, modulo comment suggestion.

Update scripts/buildtable.pl

Co-authored-by: Jon Atack <jon@atack.com>

4bbf0ffdec

jonatack commented at 4:16 PM on October 3, 2025: member

ACK. Will pull this into the typos and fix-ups compilation today.
jonatack commented at 7:25 PM on October 3, 2025: member

Cherry-picked into #1998 (thanks!), so closing here.
jonatack closed this on Oct 3, 2025

Contributors

Labels