[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r1495419 - in /subversion/trunk/subversion/libsvn_ra_serf: options.c ra_serf.h serf.c util.c

From: <kmradke_at_rockwellcollins.com>
Date: Tue, 25 Jun 2013 14:11:17 -0500

> > I agree that force-http10 is not good name and semantic. Actually
> > these proxies is not busted: it's allowed to HTTP/1.1 proxies to
> > require content-length if they want. And strictly speaking proxies may
> > have different behavior for different requests.
>
> From *our* standpoint, they are busted. Subversion wants to use
> chunked requests. If they don't support it, then they are busted.
> Simple as that.
>
> And we want to use a provocative name so that people understand
> something needs to be *fixed*. Fixed for us because we view them as
> *busted*.

From the *users* standpoint subversion is busted. Something that
worked fine in 1.7 does not work in 1.8 without modifying potentially
unrelated things that neither the server admin or the client
user may have control over. (Think proxy at a hotel, etc.)

The spec states that 411 is an allowed response and is it also states
the client should prefer to not use chunked requests if possible.
Serf is being overly optimistic here.

If this were a 2.x change it wouldn't be so concerning because there
isn't the backwards compatibility guarantee...

Kevin R.
Received on 2013-06-25 21:11:52 CEST

This is an archived mail posted to the Subversion Dev mailing list.