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

Approve group? (was: svn commit: r1502108 ...)

From: Greg Stein <gstein_at_gmail.com>
Date: Thu, 11 Jul 2013 01:52:07 -0400

Hey all,

I'd like to see us get this approved and applied to 1.8.x. My recent
changes "should" remove danielsh's issue with the patch group.

Regarding Ben's, I'd suggest: any additional change would build upon
this group. Thus, since the darned group is getting pretty large, I
think it would make sense to get it folded in.

It provides *a* solution to the proxy/chunked issue. Is more work
needed? Maybe. There isn't clear consensus yet. But again: this group
is a necessary precondition to any further work anyways.

Cheers,
-g

On Thu, Jul 11, 2013 at 12:13 AM, <gstein_at_apache.org> wrote:
> Author: gstein
> Date: Thu Jul 11 04:13:47 2013
> New Revision: 1502108
>
> URL: http://svn.apache.org/r1502108
> Log:
> update for the 1.8.x-busted-proxy branch
>
> Modified:
> subversion/branches/1.8.x/STATUS
>
> Modified: subversion/branches/1.8.x/STATUS
> URL: http://svn.apache.org/viewvc/subversion/branches/1.8.x/STATUS?rev=1502108&r1=1502107&r2=1502108&view=diff
> ==============================================================================
> --- subversion/branches/1.8.x/STATUS (original)
> +++ subversion/branches/1.8.x/STATUS Thu Jul 11 04:13:47 2013
> @@ -83,7 +83,8 @@ Candidate changes:
> Veto-blocked changes:
> =====================
>
> - * r1489117, r1496470, r1497975, r1497980, r1498012, r1499423, r1499595
> + * r1489117, r1496470, r1497975, r1497980, r1498012, r1499423, r1499595,
> + r1502097 (and local: r1499225, r1502099, r1502104, r1502106)
> Support for HTTP/1.1 reverse proxies that require Content-Length headers
> and deny chunked requests.
> Branch: ^/subversion/branches/1.8.x-busted-proxy
> @@ -104,12 +105,17 @@ Veto-blocked changes:
> is within the server-side reverse proxy
> r1499423: fix 411 handling
> r1499595: don't probe when a redirect occurs
> + r1502097: uses consensus option name from the dev@ list
>
> Local to branch:
> r1499225: remove SVN_CONFIG_OPTION_BUSTED_PROXY from the public API,
> since we cannot add a symbol in a patch release.
> + r1502099: revert r1499225
> + r1502104: similar to -9225: switch to a private symbol
> + r1502106: a simple catchup merge from 1.8.x
> Votes:
> - +1: gstein, philip, lgo
> + +1: gstein
> + +1: philip, lgo (except for r1502097, -2099, -2104, -2106)
> -1: breser (given danielsh's veto, the discussion on IRC and my email to
> dev I think we need to discuss this more before approving)
> -1: danielsh (option name is wrong. Those proxies follow the spec, so they
>
>
Received on 2013-07-11 07:52:49 CEST

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

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.