Allow environment variables to contain numbers

pizaCat · pizaCat · commit 20c73df79c9a · 2024-04-30T18:08:44.000-04:00
diff --git a/.changes/1481.json b/.changes/1481.json
@@ -0,0 +1,4 @@
+{
+    "description": "allow pass-through environment variables to contain numbers",
+    "type": "fixed"
+}
diff --git a/src/docker/shared.rs b/src/docker/shared.rs
@@ -914,7 +914,7 @@ fn validate_env_var<'a>(
         && !*warned
         && !var
             .chars()
-            .all(|c| matches!(c, 'a'..='z' | 'A'..='Z' | '_' ))
+            .all(|c| matches!(c, 'a'..='z' | 'A'..='Z' | '_' | '0'..='9'))
     {
         msg_info.warn(format_args!(
             "got {var_type} of \"{var}\" which is not a valid environment variable name. the proper syntax is {var_syntax}"

-Original file line number
+Diff line change
@@ @@ -0,0 +1,4 @@ @@
 +{
 +    "description": "allow pass-through environment variables to contain numbers",
 +    "type": "fixed"
 +}
Original file line number	Diff line number	Diff line change
`@@ -914,7 +914,7 @@ fn validate_env_var<'a>(`
`914`	`914`	`&& !*warned`
`915`	`915`	`&& !var`
`916`	`916`	`.chars()`
`917`		`- .all(\|c\| matches!(c, 'a'..='z' \| 'A'..='Z' \| '_' ))`
	`917`	`+ .all(\|c\| matches!(c, 'a'..='z' \| 'A'..='Z' \| '_' \| '0'..='9'))`
`918`	`918`	`{`
`919`	`919`	`msg_info.warn(format_args!(`
`920`	`920`	`"got {var_type} of \"{var}\" which is not a valid environment variable name. the proper syntax is {var_syntax}"`