]> git.ipfire.org Git - thirdparty/freeswitch.git/commit
Allow setting CURL timeout from curl API command
authorTravis Cross <tc@traviscross.com>
Sun, 16 Nov 2014 19:20:07 +0000 (19:20 +0000)
committerTravis Cross <tc@traviscross.com>
Sun, 16 Nov 2014 19:31:00 +0000 (19:31 +0000)
commitf1df8d609672094f3ce10a1a7de8b59a86d3db92
tree836762dc77917294abe00e919770729bd0776e98
parent1ee325df48b3c71e8cf645aaf19138eae9a43ff6
Allow setting CURL timeout from curl API command

Previously the `timeout` option to the curl API command set only
`CURLOPT_CONNECTTIMEOUT` -- the maximum amount of time that curl will
wait to connect to the server.  If the server accepted the connection
but then never replied, curl would wait essentially forever.  There
was no way to set `CURLOPT_TIMEOUT` -- the maximum amount of time the
entire request operation is allowed to take.

With this change, the `timeout` option sets `CURLOPT_TIMEOUT`.  We've
earlier added a `connect-timeout` option to set
`CURLOPT_CONNECTTIMEOUT`.

This is a change to existing behavior.  However, it's likely that this
is what people expected it to do all along.  The curl application
call, for example, accepts both `curl_connect_timeout` and
`curl_timeout` channel variables, with the latter setting
`CURLOPT_TIMEOUT`.

If people really were relying on this odd behavior, we'll rename the
option with the new behavior to something else and come up with a
transition plan.
src/mod/applications/mod_curl/mod_curl.c