Skip to content

fix: propagate http_timeout_seconds to oauth requests to prevent hangs#1480

Open
gourabsingha1 wants to merge 1 commit into
databricks:mainfrom
gourabsingha1:fix/oauth-hangs-timeout
Open

fix: propagate http_timeout_seconds to oauth requests to prevent hangs#1480
gourabsingha1 wants to merge 1 commit into
databricks:mainfrom
gourabsingha1:fix/oauth-hangs-timeout

Conversation

@gourabsingha1

Copy link
Copy Markdown

Closes #1338. Propagates Config.http_timeout_seconds down to OAuth request calls inside oauth.py, avoiding indefinite hangs when OAuth/M2M endpoints stall.

@github-actions

Copy link
Copy Markdown

If integration tests don't run automatically, an authorized user can run them manually by following the instructions below:

Trigger:
go/deco-tests-run/sdk-py

Inputs:

  • PR number: 1480
  • Commit SHA: b5943090cf07ced5b7c3bb188a6efd4ae7952b74

Checks will be approved automatically on success.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[ISSUE] requests.post() and requests.get() calls in oauth.py have no timeout, can hang indefinitely

1 participant