Merge pull request #41 from sjaensch/dont-follow-redirects

Support new request param follow_redirects, do not follow them by default

bravado_asyncio/http_client.py

@@ -185,6 +185,8 @@ def request(
             else None
         )
 
+        follow_redirects = request_params.get("follow_redirects", False)
+
         # aiohttp always adds a Content-Type header, and this breaks some servers that don't
         # expect it for non-POST/PUT requests: https://github.com/aio-libs/aiohttp/issues/457
         skip_auto_headers = (
@@ -203,6 +205,7 @@ def request(
                 k: from_bytes(v) if isinstance(v, bytes) else str(v)
                 for k, v in request_params.get("headers", {}).items()
             },
+            allow_redirects=follow_redirects,
             skip_auto_headers=skip_auto_headers,
             timeout=timeout,
             **self._get_ssl_params()

requirements-dev.txt

@@ -1,7 +1,7 @@
 aiohttp<3.6
 bottle
 bravado-core>=4.11.0
-bravado[integration-tests]>=10.4.1
+bravado[integration-tests,fido]>=10.4.1
 coverage
 ephemeral_port_reserve
 mock<4

tests/http_client_test.py

@@ -75,6 +75,7 @@ def test_request(asyncio_client, mock_client_session, request_params):
         params=None,
         data=mock.ANY,
         headers={},
+        allow_redirects=False,
         skip_auto_headers=["Content-Type"],
         ssl=None,
         timeout=None,
@@ -107,6 +108,7 @@ def test_simple_get(asyncio_client, mock_client_session, request_params):
         params=request_params["params"],
         data=mock.ANY,
         headers={},
+        allow_redirects=False,
         skip_auto_headers=["Content-Type"],
         ssl=None,
         timeout=None,
@@ -151,6 +153,7 @@ def test_formdata(
         params=None,
         data=mock.ANY,
         headers={},
+        allow_redirects=False,
         skip_auto_headers=["Content-Type"],
         ssl=None,
         timeout=None,