Azure / Azure/azure-sdk-for-cpp

CurlConnection::ReadFromSocket uses fixed 60s timeout instead of using configured timeout from HttpResponseTimeout

Open
#6,812 0 comments 0 reactions 0 assignees View on GitHub
Azure.Core customer-reported needs-triage question
Dominant language
C++
Stars
205
Forks
172
Avg merge
1d 3h
Merged PRs (30d)
37

Description

Calling GET endpoint that takes more than 60s to respond, results to "Timeout waiting for socket to read." error.

There is a fixed timeout when waiting for the socket in curl.cpp:

// start polling operation
auto pollUntilSocketIsReady = pollSocketUntilEventOrTimeout(
context, m_curlSocket, PollSocketDirection::Read, 60000L);

It would be better to have this configurable, for example from the context HttpResponseTimeout.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.