diff --git a/python-ecosys/requests/requests/__init__.py b/python-ecosys/requests/requests/__init__.py index a9a183619..2951035f7 100644 --- a/python-ecosys/requests/requests/__init__.py +++ b/python-ecosys/requests/requests/__init__.py @@ -46,6 +46,8 @@ def request( ): if headers is None: headers = {} + else: + headers = headers.copy() redirect = None # redirection url, None means no redirection chunked_data = data and getattr(data, "__next__", None) and not getattr(data, "__len__", None) diff --git a/python-ecosys/requests/test_requests.py b/python-ecosys/requests/test_requests.py index 513e533a3..ac77291b0 100644 --- a/python-ecosys/requests/test_requests.py +++ b/python-ecosys/requests/test_requests.py @@ -102,11 +102,11 @@ def chunks(): def test_overwrite_get_headers(): response = requests.request( - "GET", "http://example.com", headers={"Connection": "keep-alive", "Host": "test.com"} + "GET", "http://example.com", headers={"Host": "test.com", "Connection": "keep-alive"} ) assert response.raw._write_buffer.getvalue() == ( - b"GET / HTTP/1.0\r\n" + b"Host: test.com\r\n" + b"Connection: keep-alive\r\n\r\n" + b"GET / HTTP/1.0\r\n" + b"Connection: keep-alive\r\n" + b"Host: test.com\r\n\r\n" ), format_message(response) @@ -145,6 +145,14 @@ def chunks(): ), format_message(response) +def test_do_not_modify_headers_argument(): + global do_not_modify_this_dict + do_not_modify_this_dict = {} + requests.request("GET", "http://example.com", headers=do_not_modify_this_dict) + + assert do_not_modify_this_dict == {}, do_not_modify_this_dict + + test_simple_get() test_get_auth() test_get_custom_header() @@ -153,3 +161,4 @@ def chunks(): test_overwrite_get_headers() test_overwrite_post_json_headers() test_overwrite_post_chunked_data_headers() +test_do_not_modify_headers_argument()