time values should divided by 1000 as some versions of curl return time in μs, not s

reorx / httpstat

curl statistics made simple

MIT License

5.94k stars 384 forks source link

time values should divided by 1000 as some versions of curl return time in μs, not s #40

Closed 9506981 closed 3 years ago

9506981 commented 3 years ago

My curl version is 7.74.0 curl 7.74.0 (x86_64-pc-win32) libcurl/7.74.0 OpenSSL/1.1.1i (Schannel) zlib/1.2.11 brotli/1.0.9 zstd/1.4.8 WinIDN libssh2/1.9.0 nghttp2/1.42.0 Release-Date: 2020-12-09 Protocols: dict file ftp ftps gopher http https imap imaps ldap ldaps mqtt pop3 pop3s rtsp scp sftp smb smbs smtp smtps telnet tftp Features: AsynchDNS HSTS HTTP2 HTTPS-proxy IDN IPv6 Kerberos Largefile MultiSSL NTLM SPNEGO SSL SSPI TLS-SRP Unicode UnixSockets alt-svc brotli libz zstd

reorx commented 3 years ago

After some investigation, I found that it was caused by curl's change on time unit after 7.61.0, I have updated your code to refine the conversion logics of time values, and added some comments to explain.

Thanks for your contribution, it really helps!