KeepAlive in http requests

[jkorf]

KeepAlive in http requests

[JKorf]

As you've pointed out @ridicoulous, because HTTP 1.1 is used the default is keep-alive anyway. With your PR changes of reusing the HttpClient it should now also be able to actually re-use the connection when making a second request.

As far is I could see there is no notable performance difference (tested some requests to Binance). The consumed time per request is mainly waiting for the server to serve up the data, not making the connection.

[JKorf]

I've tried setting the Connection header to both Close and KeepAlive, but I've seen no timing difference between the two. If you can show me any data which suggests a performance improvement when setting the header I'll be happy to change it.

[ridicoulous]

@nikkozp
I solved it via overriding ConstructRequest method in my RestClient implementation.

@JKorf requests with keep alive header should reuse existing connection, instead opening new (what can took at least 30-35ms). But according to msdn, keep alive connection is used by default if you uses HTTP 1.1
May be you didn't see any diffrenece, cause your server send Close header in responses?