Revert "accept dashes in variable names"

This reverts commit be170be. Signed-off-by: Nicolas De Loof <[email protected]>
ndeloof · 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{}