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

Use httptrace WroteHeaders instead of WroteRequest #102

Merged
merged 1 commit into from
Oct 1, 2016

Conversation

moorereason
Copy link
Contributor

When POSTing a large request, the TLS Handshake timings become very
skewed. Using WroteHeaders is about the best we can do for now until
httptrace adds additional hooks.

When POSTing a large request, the TLS Handshake timings become very
skewed. Using WroteHeaders is about the best we can do for now until
httptrace adds additional hooks.
@davecheney
Copy link
Owner

Good idea.

@moorereason
Copy link
Contributor Author

For the record:

$ echo -n 'file=' > testfile && base64 -w 0 httpstat >>testfile
$ ./httpstat -d @testfile -X POST https://requestb.in/11pbuek1                                                           

Connected to 104.31.87.204:443

HTTP/2.0 200 OK
Server: cloudflare-nginx
Cf-Ray: 2ead41a3dc571159-DFW
Content-Type: text/html; charset=utf-8
Date: Sat, 01 Oct 2016 04:44:09 GMT
Set-Cookie: __cfduid=d3d76a938181f75eb2bf9d9438d1379aa1475297048; expires=Sun, 01-Oct-17 04:44:08 GMT; path=/; domain=.requestb.in; HttpOnly
Sponsored-By: https://www.runscope.com
Via: 1.1 vegur

Body discarded

  DNS Lookup   TCP Connection   TLS Handshake   Server Processing   Content Transfer
[     18ms  |          16ms  |      13407ms  |           1726ms  |             0ms  ]
            |                |               |                   |                  |
   namelookup:18ms           |               |                   |                  |
                       connect:35ms          |                   |                  |
                                   pretransfer:13443ms           |                  |
                                                     starttransfer:15169ms          |
                                                                                total:15170ms  
$ ./httpstat-wroteheaders -d @testfile -X POST https://requestb.in/11pbuek1                                                                                                               

Connected to 104.31.86.204:443

HTTP/2.0 200 OK
Server: cloudflare-nginx
Cf-Ray: 2ead4297f9671fb8-DFW
Content-Type: text/html; charset=utf-8
Date: Sat, 01 Oct 2016 04:44:49 GMT
Set-Cookie: __cfduid=db110dc6a7ba3c0a4f3618f4a757f301a1475297087; expires=Sun, 01-Oct-17 04:44:47 GMT; path=/; domain=.requestb.in; HttpOnly
Sponsored-By: https://www.runscope.com
Via: 1.1 vegur

Body discarded

  DNS Lookup   TCP Connection   TLS Handshake   Server Processing   Content Transfer
[     17ms  |          12ms  |        137ms  |          15898ms  |             0ms  ]
            |                |               |                   |                  |
   namelookup:17ms           |               |                   |                  |
                       connect:29ms          |                   |                  |
                                   pretransfer:166ms             |                  |
                                                     starttransfer:16064ms          |
                                                                                total:16065ms  

@davecheney davecheney merged commit d5cbd23 into davecheney:master Oct 1, 2016
@moorereason moorereason deleted the wroteheaders branch October 1, 2016 05:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants