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.

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

--rate does not seem to work since 7.87 #10357

Closed
highmtworks opened this issue Jan 29, 2023 · 0 comments
Closed

--rate does not seem to work since 7.87 #10357

highmtworks opened this issue Jan 29, 2023 · 0 comments

Comments

@highmtworks
Copy link

I did this

$ curl -Ss -o /dev/null https://curl.se/#[1-10] --rate 1/s -w "%header{date}\n"
Sun, 29 Jan 2023 07:58:37 GMT
Sun, 29 Jan 2023 07:58:37 GMT
Sun, 29 Jan 2023 07:58:37 GMT
Sun, 29 Jan 2023 07:58:37 GMT
Sun, 29 Jan 2023 07:58:37 GMT
Sun, 29 Jan 2023 07:58:37 GMT
Sun, 29 Jan 2023 07:58:37 GMT
Sun, 29 Jan 2023 07:58:37 GMT
Sun, 29 Jan 2023 07:58:37 GMT
Sun, 29 Jan 2023 07:58:37 GMT

I expected the following

$ pacman -U /var/cache/pacman/pkg/curl-7.86.0-1-x86_64.pkg.tar.zst

$ curl --version
curl 7.86.0 (x86_64-pc-cygwin) libcurl/7.87.0 OpenSSL/3.0.7 zlib/1.2.13 brotli/1.0.9 zstd/1.5.2 libidn2/2.3.4 libpsl/0.21.2 (+libidn2/2.3.4) libssh2/1.10.0 nghttp2/1.51.0
Release-Date: 2022-10-26
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 brotli GSS-API HSTS HTTP2 HTTPS-proxy IDN IPv6 Kerberos Largefile libz NTLM NTLM_WB PSL SPNEGO SSL threadsafe TLS-SRP UnixSockets zstd
WARNING: curl and libcurl versions do not match. Functionality may be affected.

$ curl -Ss -o /dev/null https://curl.se/#[1-10] --rate 1/s -w "%header{date}\n"
Sun, 29 Jan 2023 07:59:20 GMT
Sun, 29 Jan 2023 07:59:21 GMT
Sun, 29 Jan 2023 07:59:22 GMT
Sun, 29 Jan 2023 07:59:23 GMT
Sun, 29 Jan 2023 07:59:24 GMT
Sun, 29 Jan 2023 07:59:25 GMT
Sun, 29 Jan 2023 07:59:26 GMT
Sun, 29 Jan 2023 07:59:27 GMT
Sun, 29 Jan 2023 07:59:28 GMT
Sun, 29 Jan 2023 07:59:29 GMT

(I could not downgrade libcurl due to its dependencies .)

curl/libcurl version

$ curl --version
curl 7.87.0 (x86_64-pc-cygwin) libcurl/7.87.0 OpenSSL/3.0.7 zlib/1.2.13 brotli/1.0.9 zstd/1.5.2 libidn2/2.3.4 libpsl/0.21.2 (+libidn2/2.3.4) libssh2/1.10.0 nghttp2/1.51.0
Release-Date: 2022-12-21
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 brotli GSS-API HSTS HTTP2 HTTPS-proxy IDN IPv6 Kerberos Largefile libz NTLM NTLM_WB PSL SPNEGO SSL threadsafe TLS-SRP UnixSockets zstd

operating system

  • Windows 10 Pro 22H2 19045.2486
$ uname -a
MSYS_NT-10.0-19045 (my machine name) 3.4.5.x86_64 2023-01-20 09:17 UTC x86_64 Msys

I guess
a55256c#diff-1fa338b6a8d4b9a9ffd2243527ac4951c2576cb32dc28440f36f4f59e60643e2L2432-R2431
may be causing this ...
( per->start is set in pre_transfer, but after per = del_per_transfer(per), per->start seems to be that of the next per (i.e. zero) ? )

@bagder bagder self-assigned this Jan 29, 2023
bagder added a commit that referenced this issue Jan 29, 2023
Regression from a55256c (7.87.0)
Reported-by: highmtworks on github
Fixes #10357
@bagder bagder closed this as completed in e7512e7 Jan 29, 2023
bch pushed a commit to bch/curl that referenced this issue Jul 19, 2023
Regression from a55256c (7.87.0)
Reported-by: highmtworks on github
Fixes curl#10357
Closes curl#10358
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging a pull request may close this issue.

2 participants