[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[gnurl] 176/222: CURLOPT_TIMEOUT.3: Clarify transfer timeout time includ
From: |
gnunet |
Subject: |
[gnurl] 176/222: CURLOPT_TIMEOUT.3: Clarify transfer timeout time includes queue time |
Date: |
Thu, 07 Nov 2019 00:11:12 +0100 |
This is an automated email from the git hooks/post-receive script.
ng0 pushed a commit to branch master
in repository gnurl.
commit ce07f0b8a1ca91301f7fc79b5b0af95ed41bfd11
Author: Jay Satiro <address@hidden>
AuthorDate: Tue Oct 15 15:12:31 2019 -0400
CURLOPT_TIMEOUT.3: Clarify transfer timeout time includes queue time
Prior to this change some users did not understand that the "request"
starts when the handle is added to the multi handle, or probably they
did not understand that some of those transfers may be queued and that
time is included in timeout.
Reported-by: Jeroen Ooms
Fixes https://github.com/curl/curl/issues/4486
Closes https://github.com/curl/curl/pull/4489
---
docs/libcurl/opts/CURLOPT_TIMEOUT.3 | 11 ++++++-----
1 file changed, 6 insertions(+), 5 deletions(-)
diff --git a/docs/libcurl/opts/CURLOPT_TIMEOUT.3
b/docs/libcurl/opts/CURLOPT_TIMEOUT.3
index 1cd122df2..6de808257 100644
--- a/docs/libcurl/opts/CURLOPT_TIMEOUT.3
+++ b/docs/libcurl/opts/CURLOPT_TIMEOUT.3
@@ -40,11 +40,12 @@ In unix-like systems, this might cause signals to be used
unless
If both \fICURLOPT_TIMEOUT(3)\fP and \fICURLOPT_TIMEOUT_MS(3)\fP are set, the
value set last will be used.
-Since this puts a hard limit for how long time a request is allowed to take,
-it has limited use in dynamic use cases with varying transfer times. You are
-then advised to explore \fICURLOPT_LOW_SPEED_LIMIT(3)\fP,
-\fICURLOPT_LOW_SPEED_TIME(3)\fP or using \fICURLOPT_PROGRESSFUNCTION(3)\fP to
-implement your own timeout logic.
+Since this option puts a hard limit on how long time a request is allowed to
+take, it has limited use in dynamic use cases with varying transfer times. That
+is especially apparent when using the multi interface, which may queue the
+transfer, and that time is included. You are advised to explore
+\fICURLOPT_LOW_SPEED_LIMIT(3)\fP, \fICURLOPT_LOW_SPEED_TIME(3)\fP or using
+\fICURLOPT_PROGRESSFUNCTION(3)\fP to implement your own timeout logic.
.SH DEFAULT
Default timeout is 0 (zero) which means it never times out during transfer.
.SH PROTOCOLS
--
To stop receiving notification emails like this one, please contact
address@hidden.
- [gnurl] 153/222: docs: make sure the --no-progress-meter docs file is in dist too, (continued)
- [gnurl] 153/222: docs: make sure the --no-progress-meter docs file is in dist too, gnunet, 2019/11/06
- [gnurl] 154/222: cirrus: Increase the git clone depth., gnunet, 2019/11/06
- [gnurl] 152/222: docs: document it as --no-progress-meter instead of the reverse, gnunet, 2019/11/06
- [gnurl] 159/222: RELEASE-NOTES: synced, gnunet, 2019/11/06
- [gnurl] 157/222: cirrus: switch off blackhole status on the freebsd CI machines, gnunet, 2019/11/06
- [gnurl] 106/222: vtls: Fix comment typo about macosx-version-min compiler flag, gnunet, 2019/11/06
- [gnurl] 136/222: docs: add note on failed handles not being counted by curl_multi_perform, gnunet, 2019/11/06
- [gnurl] 139/222: http2: move state-init from creation to pre-transfer, gnunet, 2019/11/06
- [gnurl] 144/222: RELEASE-NOTES: synced, gnunet, 2019/11/06
- [gnurl] 165/222: appveyor: add a winbuild that uses VS2017, gnunet, 2019/11/06
- [gnurl] 176/222: CURLOPT_TIMEOUT.3: Clarify transfer timeout time includes queue time,
gnunet <=
- [gnurl] 173/222: tests: line ending fixes for Windows, gnunet, 2019/11/06
- [gnurl] 179/222: configure: remove all cyassl references, gnunet, 2019/11/06
- [gnurl] 183/222: http2_recv: a closed stream trumps pause state, gnunet, 2019/11/06
- [gnurl] 184/222: appveyor: Add MSVC ARM64 build, gnunet, 2019/11/06
- [gnurl] 194/222: appveyor: make winbuilds with DEBUG=no/yes and VS 2015/2017, gnunet, 2019/11/06
- [gnurl] 164/222: socketpair: fix include and define for older TCP header systems, gnunet, 2019/11/06
- [gnurl] 172/222: tests: use proxy feature, gnunet, 2019/11/06
- [gnurl] 204/222: appveyor: publish artifacts on appveyor, gnunet, 2019/11/06
- [gnurl] 156/222: tests: use port 2 instead of 60000 for a safer non-listening port, gnunet, 2019/11/06
- [gnurl] 158/222: connect: return CURLE_OPERATION_TIMEDOUT for errno == ETIMEDOUT, gnunet, 2019/11/06