subversion-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Konstantin Kolinko <knst.koli...@gmail.com>
Subject Re: [PATCH] Have serf send Content-Length request bodies
Date Mon, 16 May 2011 22:48:20 GMT
2011/5/16 Justin Erenkrantz <justin@erenkrantz.com>:
> So...at my hotel in Ljubljana, they have a silly Squid proxy that
> doesn't understand chunked request bodies.  I know this has been a
> huge detriment for some people with serf - so, here's a patch for serf
> that will try to send C-L bodies for basic HTTP request bodies.  With
> this patch, ra_serf only sends C-L...however, the hotel's version of
> Squid (squid/2.7.STABLE9) doesn't understand chunked *response* bodies
> either.  So, it ends up "Connection: close"-ing after *every single
> request*.  Needless to say, serf's not amused - it still works just
> fine; but it's pretty slow as it has to constantly re-open TCP
> connections.  There's nothing serf can do about directly controlling
> the server-side's logic to avoid chunking responses.

HTTP/1.0 does not support keep-alive, and thus the connection will be
closed after each request. You will need HTTP/1.1 to keep the
connection open.

Maybe you are able to connect with HTTPS?


Quick look at the docs says that HTTP/1.1 is disabled in Squid 2.7 by
default and its implementation "is still incomplete" [1]. Squid 3.x
should behave better.

[1] http://www.squid-cache.org/Doc/config/server_http11/

I am not a Squid admin, but just verifying my own worries about that product.

>
> So, I'm on the fence about cleaning up and applying this patch as I
> think if you're talking to a dumb HTTP proxy, you're likely going to
> run into other serious issues.
>
> Thoughts?  -- justin
>

Best regards,
Konstantin Kolinko

Sending from dev@subversion. I see that serf-dev@ is also on CC list.

Mime
View raw message