Revert "accept dashes in variable names"

This reverts commit be170be. Signed-off-by: Nicolas De Loof <[email protected]>
compose-spec · Sep 9, 2024 · aa1db26 · aa1db26
1 parent 4f02e42
commit aa1db26
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 17 deletions.
diff --git a/dotenv/fixtures/special.env b/dotenv/fixtures/special.env
diff --git a/dotenv/godotenv_test.go b/dotenv/godotenv_test.go
@@ -691,14 +691,6 @@ func TestUTF8BOM(t *testing.T) {
 	loadEnvAndCompareValues(t, Load, envFileName, expectedValues, noopPresets)
 }
 
-func TestDash(t *testing.T) {
-	loadEnvAndCompareValues(t, Load, "fixtures/special.env", map[string]string{
-		"VAR-WITH-DASHES":      "dashes",
-		"VAR.WITH.DOTS":        "dots",
-		"VAR_WITH_UNDERSCORES": "underscores",
-	}, noopPresets)
-}
-
 func TestGetEnvFromFile(t *testing.T) {
 	wd := t.TempDir()
 	f := filepath.Join(wd, ".env")

diff --git a/dotenv/parser.go b/dotenv/parser.go
@@ -119,7 +119,7 @@ loop:
 			offset = i + 1
 			inherited = rune == '\n'
 			break loop
-		case '_', '.', '-', '[', ']':
+		case '_', '.', '[', ']':
 		default:
 			// variable name should match [A-Za-z0-9_.-]
 			if unicode.IsLetter(rune) || unicode.IsNumber(rune) {

diff --git a/dotenv/parser_test.go b/dotenv/parser_test.go
@@ -13,17 +13,17 @@ var testInput = `
 a=b
 a[1]=c
 a.propertyKey=d
-árvíztűrő-TÜKÖRFÚRÓGÉP=ÁRVÍZTŰRŐ-tükörfúrógép
+árvíztűrőTÜKÖRFÚRÓGÉP=ÁRVÍZTŰRŐtükörfúrógép
 `
 
 func TestParseBytes(t *testing.T) {
 	p := newParser()
 
 	expectedOutput := map[string]string{
-		"a":                      "b",
-		"a[1]":                   "c",
-		"a.propertyKey":          "d",
-		"árvíztűrő-TÜKÖRFÚRÓGÉP": "ÁRVÍZTŰRŐ-tükörfúrógép",
+		"a":                     "b",
+		"a[1]":                  "c",
+		"a.propertyKey":         "d",
+		"árvíztűrőTÜKÖRFÚRÓGÉP": "ÁRVÍZTŰRŐtükörfúrógép",
 	}
 
 	out := map[string]string{}