Allows mocking responses with multiple headers with the same name. Fixes #927 (#929)

waldekmastykarz · web-flow · commit 6f40ca34ecbb · 2024-11-05T10:22:48.000+01:00
diff --git a/dev-proxy-abstractions/ProxyUtils.cs b/dev-proxy-abstractions/ProxyUtils.cs
@@ -339,7 +339,11 @@ public static void MergeHeaders(IList<MockResponseHeader> allHeaders, IList<Mock
                     continue;
                 }
 
-                allHeaders.Remove(existingHeader);
+                // don't remove headers that we've just added
+                if (!headersToAdd.Contains(existingHeader))
+                {
+                    allHeaders.Remove(existingHeader);
+                }
             }
 
             allHeaders.Add(header);

Original file line number	Diff line number	Diff line change
`@@ -339,7 +339,11 @@ public static void MergeHeaders(IList<MockResponseHeader> allHeaders, IList<Mock`
`339`	`339`	`continue;`
`340`	`340`	`}`
`341`	`341`
`342`		`- allHeaders.Remove(existingHeader);`
	`342`	`+ // don't remove headers that we've just added`
	`343`	`+ if (!headersToAdd.Contains(existingHeader))`
	`344`	`+ {`
	`345`	`+ allHeaders.Remove(existingHeader);`
	`346`	`+ }`
`343`	`347`	`}`
`344`	`348`
`345`	`349`	`allHeaders.Add(header);`