subversion-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Greg Stein <gst...@gmail.com>
Subject Re: Coniguring 301/302 redirects to track an fspath rename
Date Thu, 07 Feb 2013 16:02:32 GMT
Cool. Looks good, along with my change last year.
On Feb 7, 2013 9:54 AM, "C. Michael Pilato" <cmpilato@collab.net> wrote:

> I committed this patch plus the fixes for the other problem I mentioned in
> r1443578, hopefully without breaking the workaround that Greg put into
> place
> in r1351138 for sloppy code.google.com response generation.
>
>
> On 02/05/2013 05:44 PM, C. Michael Pilato wrote:
> > On 02/05/2013 05:26 PM, Bert Huijben wrote:
> >> Looking at this stack, this error originates in the iprops fetch that
> was
> >> added before the actual update for 1.8.
> >
> > Here's the immediate fix for the NULL location:
> >
> > {{{
> > Index: subversion/libsvn_ra_serf/property.c
> > ===================================================================
> > --- subversion/libsvn_ra_serf/property.c      (revision 1442766)
> > +++ subversion/libsvn_ra_serf/property.c      (working copy)
> > @@ -637,7 +637,7 @@
> >
> >    err2 = svn_ra_serf__error_on_status(handler->sline.code,
> >                                        handler->path,
> > -                                      NULL);
> > +                                      handler->location);
> >    if (err2)
> >      {
> >        svn_error_clear(err);
> > }}}
> >
> > I haven't time to give a full test run right now -- heading out the door
> for
> > a meeting.
> >
> > But there's another problem here anyway, which is that
> > response_get_location() in libsvn_ra_serf/util.c is only return the path
> > portion of the URL provided by the server.  Not sure precisely when that
> > started happening -- r1351138 seems key here -- but either way, we
> > absolutely need the full (canonicalized-for-Subversion-use) URL here to
> > continue properly handling redirects which point to a different server.
> >
>
>
> --
> C. Michael Pilato <cmpilato@collab.net>
> CollabNet   <>   www.collab.net   <>   Enterprise Cloud Development
>
>

Mime
View raw message