Merge branch 'master' of github.com:DataDog/dd-apm-test-agent into sabrenner/remove-vcr-handling

Author: sabrenner

ddapm_test_agent/vcr_proxy.py

@@ -290,7 +290,7 @@ async def _request(
 
         response = Response(body=provider_response.content, status=provider_response.status_code)
         response_headers = provider_response.headers
-        
+
     _write_response_headers(response, response_headers)
 
     return response
@@ -329,7 +329,8 @@ async def proxy_request(
         )
 
     target_url = _url_path_join(provider_base_urls[provider], remaining_path)
-    headers = {key: value for key, value in request.headers.items() if key != "Host"}
+    skip_headers = {"host", "transfer-encoding"}
+    headers = {key: value for key, value in request.headers.items() if not (key.lower() in skip_headers)}
 
     request_kwargs: Dict[str, Any] = {
         "method": request.method,

releasenotes/notes/ignore-transfer-encoding-header-3b22699615356f91.yaml

@@ -0,0 +1,5 @@
+---
+fixes:
+  - |
+    Ignores the Transfer-Encoding header in proxied vcr requests, forwarding this header can lead to failures in 
+    downstream services. 

tests/test_vcr_proxy.py

@@ -195,3 +195,10 @@ async def test_vcr_proxy_does_not_record_ignored_headers(
     assert "User-Super-Secret-Api-Key" not in recorded_request["request"]["headers"]
     assert "Foo-Bar" not in recorded_request["request"]["headers"]
     assert "Authorization" not in recorded_request["request"]["headers"]
+
+
+async def test_vcr_proxy_with_chunked_request(agent: TestClient[Any, Any], vcr_cassettes_directory: str) -> None:
+    resp = await agent.post("/vcr/custom/serve", json={"foo": "bar"}, chunked=True)
+
+    assert resp.status == 200
+    assert await resp.text() == "OK"