Merge pull request #468 from tino/properly-escaped-contenttype-regex

Fix the regex of the example proxy.gzip.contenttype
fabiolb · Mar 18, 2018 · 029e660 · 029e660
2 parents e329e31 + bcd66de
commit 029e660
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/config/load_test.go b/config/load_test.go
@@ -425,6 +425,13 @@ func TestLoad(t *testing.T) {
 				return cfg
 			},
 		},
+		{
+			args: []string{"-proxy.gzip.contenttype", "^(text/.*|application/(javascript|json|font-woff|xml)|.*\\+(json|xml))(;.*)?$"},
+			cfg: func(cfg *Config) *Config {
+				cfg.Proxy.GZIPContentTypes = regexp.MustCompile(`^(text/.*|application/(javascript|json|font-woff|xml)|.*\+(json|xml))(;.*)?$`)
+				return cfg
+			},
+		},
 		{
 			args: []string{"-proxy.log.routes", "foobar"},
 			cfg: func(cfg *Config) *Config {

diff --git a/fabio.properties b/fabio.properties
@@ -441,7 +441,7 @@
 #
 # A typical example is
 #
-# proxy.gzip.contenttype = ^(text/.*|application/(javascript|json|font-woff|xml)|.*\+(json|xml))(;.*)?$
+# proxy.gzip.contenttype = ^(text/.*|application/(javascript|json|font-woff|xml)|.*\\+(json|xml))(;.*)?$
 #
 # The default is
 #