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

curl -w times shown as microseconds instead of seconds with fractions #6321

Closed
notsure2 opened this issue Dec 14, 2020 · 0 comments
Closed

curl -w times shown as microseconds instead of seconds with fractions #6321

notsure2 opened this issue Dec 14, 2020 · 0 comments

Comments

@notsure2
Copy link

@notsure2 notsure2 commented Dec 14, 2020

root@linux:/opt/root/bin# /opt/bin/curl --version && /opt/bin/curl -w '%{time_total}\n' -o /dev/null -s 'http://www.cnn.com/'
curl 7.72.0 (arm-openwrt-linux-gnu) libcurl/7.72.0 OpenSSL/1.1.1h zlib/1.2.11
Release-Date: 2020-08-19
Protocols: file ftp ftps http https imap imaps pop3 pop3s rtsp smtp smtps tftp
Features: HTTPS-proxy IPv6 Largefile libz SSL
0.236153

root@linux:/opt/root/bin# /usr/sbin/curl --version && /usr/sbin/curl -w '%{time_total}\n' -o /dev/null -s 'http://www.cnn.com/'
curl 7.74.0 (arm-unknown-linux-gnu) libcurl/7.74.0 OpenSSL/1.1.1i zlib/1.2.11
Release-Date: 2020-12-09
Protocols: file ftp ftps http https imap imaps mqtt pop3 pop3s rtsp smb smbs smtp smtps tftp
Features: alt-svc HTTPS-proxy IPv6 Largefile libz NTLM NTLM_WB SSL TLS-SRP UnixSockets
126886

Is this documented anywhere ? This is a MAJOR breaking change in a minor release.

@bagder bagder changed the title curl 7.74 BREAKING CHANGE to time units of format outputs without warning curl -w times shown as microseconds instead of seconds with fractions Dec 14, 2020
bagder added a commit that referenced this issue Dec 14, 2020
Fix regression from commit fc813f8 (#6248) that changed the unit
to microseconds instead of seconds with fractions

Fixes #6321
jay added a commit to jay/curl that referenced this issue Dec 14, 2020
fc813f8 (precedes 7.74.0) changed write-out to retrieve the times using
curl_off_t (microseconds) instead of double (seconds) but did not
account for that in the formatted output.

Fixes curl#6321
Closes #xxxx
@bagder bagder closed this in bc7ecc7 Dec 15, 2020
This was referenced Dec 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

2 participants
You can’t perform that action at this time.