From 8b235cd34e71e1102a9cf647706969f83bbd7642 Mon Sep 17 00:00:00 2001
From: Nicolas De Loof <nicolas.deloof@gmail.com>
Date: Thu, 5 Sep 2024 10:38:21 +0200
Subject: [PATCH] Revert "accept dashes in variable names"

This reverts commit be170befda31a336a94d248ac1221fceeac834c8.

Signed-off-by: Nicolas De Loof <nicolas.deloof@gmail.com>
---
 dotenv/fixtures/special.env |  3 ---
 dotenv/godotenv_test.go     |  8 --------
 dotenv/parser.go            |  2 +-
 dotenv/parser_test.go       | 10 +++++-----
 4 files changed, 6 insertions(+), 17 deletions(-)
 delete mode 100644 dotenv/fixtures/special.env

diff --git a/dotenv/fixtures/special.env b/dotenv/fixtures/special.env
deleted file mode 100644
index 5499b0cd..00000000
--- a/dotenv/fixtures/special.env
+++ /dev/null
@@ -1,3 +0,0 @@
-VAR.WITH.DOTS=dots
-VAR_WITH_UNDERSCORES=underscores
-VAR-WITH-DASHES=dashes
diff --git a/dotenv/godotenv_test.go b/dotenv/godotenv_test.go
index 0d77906c..84f443ef 100644
--- 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
index 861cd953..d7b4a646 100644
--- 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
index 54580e2a..6764d583 100644
--- 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{}