Skip to content

config timeout with CURLOPT_TIMEOUT_MS instead of CURLOPT_TIMEOUT for curl implementation #1258

New issue
New issue
Closed
#1261
Closed
config timeout with CURLOPT_TIMEOUT_MS instead of CURLOPT_TIMEOUT for curl implementation#1258
#1261
Labels
area:exporterbugSomething isn't workingdo-not-stalepriority:p2Issues that are not blocking
@lalitb

Description

@lalitb
lalitb
opened on Mar 10, 2022

Discussed in #1254

Originally posted by wxl374 March 10, 2022
When configure OtlpHttpExporter's timeout less than 1s, it will set to 0 for the use of CURLOPT_TIMEOUT(

opentelemetry-cpp/ext/include/opentelemetry/ext/http/client/curl/http_operation_curl.h

Line 189 in fd5a621

curl_easy_setopt(curl_, CURLOPT_TIMEOUT, http_conn_timeout_.count() / 1000);
), then never times out (sometimes limited by CURLOPT_LOW_SPEED_TIME). Why not use CURLOPT_TIMEOUT_MS instead？

Metadata

Metadata

Assignees

No one assigned

    Labels

    area:exporterbugSomething isn't workingdo-not-stalepriority:p2Issues that are not blocking

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions