Skip to content

Commit c10bee7

Browse files
ai: apply changes for #876 (1 review thread)
Addresses: - #3636633446 at src/databricks/sql/auth/oauth.py:85 Signed-off-by: peco-engineer-bot[bot] <peco-engineer-bot[bot]@users.noreply.github.com>
1 parent fbdb5dc commit c10bee7

1 file changed

Lines changed: 4 additions & 2 deletions

File tree

src/databricks/sql/auth/oauth.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,9 @@ class OAuthManager:
7373
# ``redirect_callback_timeout_seconds`` argument below is an internal-only
7474
# override for callers constructing ``OAuthManager`` directly (private API);
7575
# it is not plumbed through ``DatabricksOAuthProvider`` or the public
76-
# connection kwargs.
76+
# connection kwargs. It accepts a ``float`` so callers (and tests) may pass
77+
# fractional-second timeouts; this class default is a whole number of
78+
# seconds.
7779
REDIRECT_CALLBACK_TIMEOUT_SECONDS = 60 * 5
7880

7981
def __init__(
@@ -82,7 +84,7 @@ def __init__(
8284
client_id: str,
8385
idp_endpoint: OAuthEndpointCollection,
8486
http_client,
85-
redirect_callback_timeout_seconds: Optional[int] = None,
87+
redirect_callback_timeout_seconds: Optional[float] = None,
8688
):
8789
self.port_range = port_range
8890
self.client_id = client_id

0 commit comments

Comments
 (0)