Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

CURLOPT_TIMEOUT default is 1 hour (on TFTP) #6774

Closed
Togtja opened this issue Mar 22, 2021 · 1 comment
Closed

CURLOPT_TIMEOUT default is 1 hour (on TFTP) #6774

Togtja opened this issue Mar 22, 2021 · 1 comment
Assignees
@bagder
Labels
TFTP

Comments

@Togtja
Copy link

Togtja commented Mar 22, 2021

I did this

I used libcurl to download over a very slow (but stable) TFTP link, and it keeps timing out after 3600 seconds aka 1 hour.
Then I set the option to be 0 curl_easy_setopt(curl, CURLOPT_TIMEOUT, 0L); it still times out after 3600 seconds.
So I thought it might have to do with the server timing out after 1 hour, but when setting the CURLOPT_TIMEOUT to 4000, it times out after 4000 seconds.

I wish to timeout using the stability of the connection aka CURLOPT_LOW_SPEED_LIMIT and CURLOPT_LOW_SPEED_TIME, both of which are also set

I expected the following

According to CURLOPT_TIMEOUT and CURLOPT_TIMEOUT_MS documentation, the default should be 0 and if it is 0 it should never time out

curl/libcurl version

curl 7.75.0 (Linux) libcurl/7.75.0 OpenSSL/1.1.1g zlib/1.2.11 libssh2/1.9.0_DEV
Release-Date: 2021-02-03
Protocols: dict file ftp ftps gopher gophers http https imap imaps mqtt pop3 pop3s rtsp scp sftp smb smbs smtp smtps telnet tftp
Features: alt-svc AsynchDNS HTTPS-proxy IPv6 Largefile libz NTLM SSL UnixSockets
I also ran this on 7.74.0, same issues

operating system

Linux tomas-ubuntu 5.4.0-66-generic #74-Ubuntu SMP Wed Jan 27 22:54:38 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
@bagder bagder added the TFTP label Mar 22, 2021
@bagder bagder self-assigned this Mar 22, 2021
bagder added a commit that referenced this issue Mar 22, 2021
@bagder
tftp: remove the 3600 second default timeout
a084a21 
... it was never meant to be there.

Reported-by: Tomas Berger
Fixes #6774
Closes #[fill in]
@bagder bagder mentioned this issue Mar 22, 2021
Closed
@bagder bagder closed this as completed in d3d90ad Mar 22, 2021
@gvollant
Copy link
Contributor

#6774 / #6776 introduced windows warning, fix on #6779

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bagder bagder

Labels
TFTP
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

tftp: remove the 3600 second default timeout curl/curl
3 participants
@bagder @gvollant @Togtja