Keep Alive grade

[rviscomi]

I was trolling the public test results and came across this curious test: http://www.webpagetest.org/result/130206_J8_J8B/1/details/

The “Keep Alive Enabled” grade is A but it clearly opens a new connection for all first-party content. Curious if this is intentional or a bug.

Thanks

Edit: This test shows similar behavior: http://www.webpagetest.org/result/130206_4D_JMJ/1/details/

[hsiboy]

That test failed in some way, i’ve seen this a few times.

@hsiboy

@rviscomi

I was trolling the public test results and came across this curious test: http://www.webpagetest.org/result/130206_J8_J8B/1/details/

The “Keep Alive Enabled” grade is A but it clearly opens a new connection for all first-party content. Curious if this is intentional or a bug.

Thanks

Edit: This test shows similar behavior: http://www.webpagetest.org/result/130206_4D_JMJ/1/details/

[pmeenan]

Looks like the logic is getting confused by:

Connection: keep-alive, close

I don’t see anything in the specs that expect both to be there but clearly my code is seeing the keep-alive first and trusting it while browsers are seeing the close and honoring that.

I created an issue to track it- http://code.google.com/p/webpagetest/issues/detail?id=141