From 0da879b2fb24c7378bb30e607006d40bc95e1ce0 Mon Sep 17 00:00:00 2001 From: Matt Dzwonczyk <9063128+mattgd@users.noreply.github.com> Date: Mon, 13 Jan 2025 16:53:09 -0500 Subject: [PATCH] Don't overwrite default transport so proxies can be set via environment variables (#398) --- workos/utils/http_client.py | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/workos/utils/http_client.py b/workos/utils/http_client.py index bc8dd426..203c7df0 100644 --- a/workos/utils/http_client.py +++ b/workos/utils/http_client.py @@ -38,7 +38,9 @@ def __init__( client_id: str, version: str, timeout: Optional[int] = None, - transport: Optional[httpx.BaseTransport] = httpx.HTTPTransport(), + # If no custom transport is provided, let httpx use the default + # so we don't overwrite environment configurations like proxies + transport: Optional[httpx.BaseTransport] = None, ) -> None: super().__init__( api_key=api_key, @@ -136,7 +138,9 @@ def __init__( client_id: str, version: str, timeout: Optional[int] = None, - transport: Optional[httpx.AsyncBaseTransport] = httpx.AsyncHTTPTransport(), + # If no custom transport is provided, let httpx use the default + # so we don't overwrite environment configurations like proxies + transport: Optional[httpx.AsyncBaseTransport] = None, ) -> None: super().__init__( base_url=base_url,