From a2dab46adfea472eb7757399d33e15412e649ffc Mon Sep 17 00:00:00 2001 From: ahiggins-arti Date: Tue, 21 Apr 2026 14:48:30 -0400 Subject: [PATCH] Revert "fix(env): escape newlines in environment variable values (#300)" This reverts commit de4200d1dfa7b95202583f4e4ad5d1201364df92. --- env.go | 2 +- env_test.go | 8 -------- 2 files changed, 1 insertion(+), 9 deletions(-) diff --git a/env.go b/env.go index 9064c97..19954d0 100644 --- a/env.go +++ b/env.go @@ -57,7 +57,7 @@ func (e *EnvMap) Environ() []string { }) return lo.MapToSlice(env, func(k string, v string) string { - return fmt.Sprintf("%s=%s", k, strings.ReplaceAll(v, "\n", "\\n")) + return fmt.Sprintf("%s=%s", k, v) }) } diff --git a/env_test.go b/env_test.go index 56fecde..d548d30 100644 --- a/env_test.go +++ b/env_test.go @@ -25,12 +25,6 @@ func TestEnvMap(t *testing.T) { e.Add("DOCKER_CONSUL_BOOTSTRAP_EXPAND_ONE", "foo-${DOCKER_CONSUL_BOOTSTRAP_TEST_FOO}-bar") e.Add("DOCKER_CONSUL_BOOTSTRAP_EXPAND_TWO", "foo-${DOCKER_CONSUL_BOOTSTRAP_TEST_BAZ}-baz") e.Add(" ", "test-empty") - e.Add("DOCKER_CONSUL_BOOTSTRAP_NEWLINE_ESCAPED", "foo\\n\\nbar") - e.Add("DOCKER_CONSUL_BOOTSTRAP_NEWLINE", `foo -bar -baz - -test`) assert.ElementsMatch(t, []string{ "DOCKER_CONSUL_BOOTSTRAP_TEST_TEST=testing", @@ -39,7 +33,5 @@ test`) "DOCKER_CONSUL_BOOTSTRAP_TEST_BAZ=bar", "DOCKER_CONSUL_BOOTSTRAP_EXPAND_ONE=foo-ignore-bar", "DOCKER_CONSUL_BOOTSTRAP_EXPAND_TWO=foo-bar-baz", - "DOCKER_CONSUL_BOOTSTRAP_NEWLINE_ESCAPED=foo\\n\\nbar", - "DOCKER_CONSUL_BOOTSTRAP_NEWLINE=foo\\nbar\\nbaz\\n\\ntest", }, e.Environ()) }