Skip to content

Configurable http timeout and http response header timeout for azurebs - #183

Open
jochenehret wants to merge 5 commits into
mainfrom
azure_http_timeouts_2
Open

jochenehret wants to merge 5 commits into
mainfrom
azure_http_timeouts_2

Conversation

@jochenehret

@jochenehret jochenehret commented Sep 9, 2026

Copy link
Copy Markdown
Contributor
  • copies http transport settings from azure-sdk-for-go (runtime/transport_default_http_client.go)
  • integration tests ensures that a short timeout is respected

Comment thread azurebs/client/storage_client.go Outdated
}

// preserve the default transport settings from the azure-sdk-for-go runtime package
transport := http.DefaultTransport.(*http.Transport).Clone()

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The transport is based on http.DefaultTransport instead of the default transport provided by the Azure SDK which has a tuned configuration (TLS, timeouts, connection pool config).

See azcore/runtime/transport_default_http_client.go.

@github-project-automation github-project-automation Bot moved this from Inbox to Waiting for Changes | Open for Contribution in Foundational Infrastructure Working Group Sep 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Waiting for Changes | Open for Contribution

Development

Successfully merging this pull request may close these issues.

2 participants