domdoescode
diff --git a/‎caddyhttp/proxy/proxy.go‎
Lines changed: 8 additions & 2 deletions b/‎caddyhttp/proxy/proxy.go‎
Lines changed: 8 additions & 2 deletions
diff --git a/‎caddyhttp/proxy/proxy_test.go‎
Lines changed: 12 additions & 0 deletions b/‎caddyhttp/proxy/proxy_test.go‎
Lines changed: 12 additions & 0 deletions
@@ -287,12 +287,18 @@ func mutateHeadersByRules(headers, rules http.Header, repl httpserver.Replacer)
 	for ruleField, ruleValues := range rules {
 		if strings.HasPrefix(ruleField, "+") {
 			for _, ruleValue := range ruleValues {
-				headers.Add(strings.TrimPrefix(ruleField, "+"), repl.Replace(ruleValue))
+				replacement := repl.Replace(ruleValue)
+				if len(replacement) > 0 {
+					headers.Add(strings.TrimPrefix(ruleField, "+"), replacement)
+				}
 			}
 		} else if strings.HasPrefix(ruleField, "-") {
 			headers.Del(strings.TrimPrefix(ruleField, "-"))
 		} else if len(ruleValues) > 0 {
-			headers.Set(ruleField, repl.Replace(ruleValues[len(ruleValues)-1]))
+			replacement := repl.Replace(ruleValues[len(ruleValues)-1])
+			if len(replacement) > 0 {
+				headers.Set(ruleField, replacement)
+			}
 		}
 	}
 }
@@ -407,8 +407,10 @@ func TestUpstreamHeadersUpdate(t *testing.T) {
 		"Upgrade":    {"{>Upgrade}"},
 		"+Merge-Me":  {"Merge-Value"},
 		"+Add-Me":    {"Add-Value"},
+		"+Add-Empty": {"{}"},
 		"-Remove-Me": {""},
 		"Replace-Me": {"{hostname}"},
+		"Clear-Me":   {""},
 		"Host":       {"{>Host}"},
 	}
 	// set up proxy
@@ -448,6 +450,11 @@ func TestUpstreamHeadersUpdate(t *testing.T) {
 			headerKey, expect, got)
 	}
 
+	headerKey = "Add-Empty"
+	if _, ok := actualHeaders[headerKey]; ok {
+		t.Errorf("Request sent to upstream backend should not contain empty %v header", headerKey)
+	}
+
 	headerKey = "Remove-Me"
 	if _, ok := actualHeaders[headerKey]; ok {
 		t.Errorf("Request sent to upstream backend should not contain %v header", headerKey)
@@ -461,6 +468,11 @@ func TestUpstreamHeadersUpdate(t *testing.T) {
 			headerKey, expect, got)
 	}
 
+	headerKey = "Clear-Me"
+	if _, ok := actualHeaders[headerKey]; ok {
+		t.Errorf("Request sent to upstream backend should not contain empty %v header", headerKey)
+	}
+
 	if actualHost != expectHost {
 		t.Errorf("Request sent to upstream backend should have value of Host with %s, but got %s", expectHost, actualHost)
 	}
Original file line number	Diff line number	Diff line change
@@ -287,12 +287,18 @@ func mutateHeadersByRules(headers, rules http.Header, repl httpserver.Replacer)
287	287	for ruleField, ruleValues := range rules {
288	288	if strings.HasPrefix(ruleField, "+") {
289	289	for _, ruleValue := range ruleValues {
290		- headers.Add(strings.TrimPrefix(ruleField, "+"), repl.Replace(ruleValue))
	290	+ replacement := repl.Replace(ruleValue)
	291	+ if len(replacement) > 0 {
	292	+ headers.Add(strings.TrimPrefix(ruleField, "+"), replacement)
	293	+ }
291	294	}
292	295	} else if strings.HasPrefix(ruleField, "-") {
293	296	headers.Del(strings.TrimPrefix(ruleField, "-"))
294	297	} else if len(ruleValues) > 0 {
295		- headers.Set(ruleField, repl.Replace(ruleValues[len(ruleValues)-1]))
	298	+ replacement := repl.Replace(ruleValues[len(ruleValues)-1])
	299	+ if len(replacement) > 0 {
	300	+ headers.Set(ruleField, replacement)
	301	+ }
296	302	}
297	303	}
298	304	}