[prev in list] [next in list] [prev in thread] [next in thread] 

List:       subversion-commits
Subject:    Re: svn commit: r1655821 - in /subversion/trunk/subversion: libsvn_ra_serf/options.c tests/cmdline/r
From:       Branko Čibej <brane () wandisco ! com>
Date:       2015-01-29 20:22:10
Message-ID: 54CA9672.5080806 () wandisco ! com
[Download RAW message or body]

On 29.01.2015 20:27, philip@apache.org wrote:
> Author: philip
> Date: Thu Jan 29 19:27:19 2015
> New Revision: 1655821
> 
> URL: http://svn.apache.org/r1655821
> Log:
> Distingush between permanent and temporary redirections to match the
> 1.8 behaviour.
> 
> * subversion/libsvn_ra_serf/options.c
> (svn_ra_serf__exchange_capabilities): Permanent or temporary error.
> 
> * subversion/tests/cmdline/redirect_tests.py
> (redirected_copy): Extend with temporary redirect.
> 
> Modified:
> subversion/trunk/subversion/libsvn_ra_serf/options.c
> subversion/trunk/subversion/tests/cmdline/redirect_tests.py
> 
> Modified: subversion/trunk/subversion/libsvn_ra_serf/options.c
> URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_ra_serf/options.c?rev=1655821&r1=1655820&r2=1655821&view=diff
>  ==============================================================================
> --- subversion/trunk/subversion/libsvn_ra_serf/options.c (original)
> +++ subversion/trunk/subversion/libsvn_ra_serf/options.c Thu Jan 29 19:27:19 2015
> @@ -555,8 +555,9 @@ svn_ra_serf__exchange_capabilities(svn_r
> && opt_ctx->handler->sline.code < 399)
> {
> return svn_error_createf(SVN_ERR_RA_SESSION_URL_MISMATCH, NULL,
> -                              _("The repository reports that it was moved "
> -                                "to '%s'"),
> +                               (opt_ctx->handler->sline.code == 301
> +                                ? _("Repository moved permanently to '%s'")
> +                                : _("Repository moved temporarily to '%s'")),
> opt_ctx->handler->location);
> }

I just noticed that we're probably handling the 307 status incorrectly,
or not at all.

-- Brane


[prev in list] [next in list] [prev in thread] [next in thread]