Skip to content

Commit 4d47f85

Browse files
feat(client): add follow_redirects request option
1 parent 187ecc9 commit 4d47f85

File tree

4 files changed

+64
-0
lines changed

4 files changed

+64
-0
lines changed

src/ydc_search_api/_base_client.py

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -960,6 +960,9 @@ def request(
960960
if self.custom_auth is not None:
961961
kwargs["auth"] = self.custom_auth
962962

963+
if options.follow_redirects is not None:
964+
kwargs["follow_redirects"] = options.follow_redirects
965+
963966
log.debug("Sending HTTP Request: %s %s", request.method, request.url)
964967

965968
response = None
@@ -1460,6 +1463,9 @@ async def request(
14601463
if self.custom_auth is not None:
14611464
kwargs["auth"] = self.custom_auth
14621465

1466+
if options.follow_redirects is not None:
1467+
kwargs["follow_redirects"] = options.follow_redirects
1468+
14631469
log.debug("Sending HTTP Request: %s %s", request.method, request.url)
14641470

14651471
response = None

src/ydc_search_api/_models.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -737,6 +737,7 @@ class FinalRequestOptionsInput(TypedDict, total=False):
737737
idempotency_key: str
738738
json_data: Body
739739
extra_json: AnyMapping
740+
follow_redirects: bool
740741

741742

742743
@final
@@ -750,6 +751,7 @@ class FinalRequestOptions(pydantic.BaseModel):
750751
files: Union[HttpxRequestFiles, None] = None
751752
idempotency_key: Union[str, None] = None
752753
post_parser: Union[Callable[[Any], Any], NotGiven] = NotGiven()
754+
follow_redirects: Union[bool, None] = None
753755

754756
# It should be noted that we cannot use `json` here as that would override
755757
# a BaseModel method in an incompatible fashion.

src/ydc_search_api/_types.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -100,6 +100,7 @@ class RequestOptions(TypedDict, total=False):
100100
params: Query
101101
extra_json: AnyMapping
102102
idempotency_key: str
103+
follow_redirects: bool
103104

104105

105106
# Sentinel class used until PEP 0661 is accepted
@@ -215,3 +216,4 @@ class _GenericAlias(Protocol):
215216

216217
class HttpxSendArgs(TypedDict, total=False):
217218
auth: httpx.Auth
219+
follow_redirects: bool

tests/test_client.py

Lines changed: 54 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -818,6 +818,33 @@ def retry_handler(_request: httpx.Request) -> httpx.Response:
818818

819819
assert response.http_request.headers.get("x-stainless-retry-count") == "42"
820820

821+
@pytest.mark.respx(base_url=base_url)
822+
def test_follow_redirects(self, respx_mock: MockRouter) -> None:
823+
# Test that the default follow_redirects=True allows following redirects
824+
respx_mock.post("/redirect").mock(
825+
return_value=httpx.Response(302, headers={"Location": f"{base_url}/redirected"})
826+
)
827+
respx_mock.get("/redirected").mock(return_value=httpx.Response(200, json={"status": "ok"}))
828+
829+
response = self.client.post("/redirect", body={"key": "value"}, cast_to=httpx.Response)
830+
assert response.status_code == 200
831+
assert response.json() == {"status": "ok"}
832+
833+
@pytest.mark.respx(base_url=base_url)
834+
def test_follow_redirects_disabled(self, respx_mock: MockRouter) -> None:
835+
# Test that follow_redirects=False prevents following redirects
836+
respx_mock.post("/redirect").mock(
837+
return_value=httpx.Response(302, headers={"Location": f"{base_url}/redirected"})
838+
)
839+
840+
with pytest.raises(APIStatusError) as exc_info:
841+
self.client.post(
842+
"/redirect", body={"key": "value"}, options={"follow_redirects": False}, cast_to=httpx.Response
843+
)
844+
845+
assert exc_info.value.response.status_code == 302
846+
assert exc_info.value.response.headers["Location"] == f"{base_url}/redirected"
847+
821848

822849
class TestAsyncYdcSearchAPI:
823850
client = AsyncYdcSearchAPI(base_url=base_url, api_key=api_key, _strict_response_validation=True)
@@ -1639,3 +1666,30 @@ async def test_main() -> None:
16391666
raise AssertionError("calling get_platform using asyncify resulted in a hung process")
16401667

16411668
time.sleep(0.1)
1669+
1670+
@pytest.mark.respx(base_url=base_url)
1671+
async def test_follow_redirects(self, respx_mock: MockRouter) -> None:
1672+
# Test that the default follow_redirects=True allows following redirects
1673+
respx_mock.post("/redirect").mock(
1674+
return_value=httpx.Response(302, headers={"Location": f"{base_url}/redirected"})
1675+
)
1676+
respx_mock.get("/redirected").mock(return_value=httpx.Response(200, json={"status": "ok"}))
1677+
1678+
response = await self.client.post("/redirect", body={"key": "value"}, cast_to=httpx.Response)
1679+
assert response.status_code == 200
1680+
assert response.json() == {"status": "ok"}
1681+
1682+
@pytest.mark.respx(base_url=base_url)
1683+
async def test_follow_redirects_disabled(self, respx_mock: MockRouter) -> None:
1684+
# Test that follow_redirects=False prevents following redirects
1685+
respx_mock.post("/redirect").mock(
1686+
return_value=httpx.Response(302, headers={"Location": f"{base_url}/redirected"})
1687+
)
1688+
1689+
with pytest.raises(APIStatusError) as exc_info:
1690+
await self.client.post(
1691+
"/redirect", body={"key": "value"}, options={"follow_redirects": False}, cast_to=httpx.Response
1692+
)
1693+
1694+
assert exc_info.value.response.status_code == 302
1695+
assert exc_info.value.response.headers["Location"] == f"{base_url}/redirected"

0 commit comments

Comments
 (0)