ConnState hook for StateActive not working

[alxarch]

It seems that the ConnState hook is not called for a connection transitioning to StateActive. The hook call is inside an if that never evaluates to true since RequestHeader.tryRead() discards the header bytes before returning thus if r.Buffered() > 0 is always false for requests with no body.

[alxarch]

Seems to be working OK with if err == nil here but i don't know if there are corner cases where a non nil err results in an non active connection.

[erikdubbelboer]

I think only setting it if err == nil should work. As you can see below it will always break out of the request loop if err != nil so it will never transition into an active connection state if err != nil.

[erikdubbelboer]

Should be fixed in 1b3ac0c