Re: svn commit: r1655435 - in /subversion/trunk/subversion: libsvn_ra_serf/options.c libsvn_ra_serf/ra_serf.h libsvn_ra_serf/util.c tests/cmdline/redirect_tests.py

[Philip Martin]

rhuij...@apache.org writes:

> --- subversion/trunk/subversion/libsvn_ra_serf/util.c (original)
> +++ subversion/trunk/subversion/libsvn_ra_serf/util.c Wed Jan 28 19:16:05 2015
> @@ -1860,6 +1860,12 @@ response_done(serf_request_t *request,
>        return svn_error_trace(svn_ra_serf__unexpected_status(handler));
>      }
>  
> +  if ((handler->sline.code >= 300 && handler->sline.code < 399)
> +      && !handler->no_fail_on_http_redirect_status)
> +    {
> +      return svn_error_trace(svn_ra_serf__unexpected_status(handler));
> +    }
> +
>    return SVN_NO_ERROR;
>  }

That ignores handler->session->pending_error while immediately above
this we have:

  if ((handler->sline.code >= 400 || handler->sline.code <= 199)
      && !handler->session->pending_error
      && !handler->no_fail_on_http_failure_status)
    {
      return svn_error_trace(svn_ra_serf__unexpected_status(handler));
    }

Why is the handling different?

-- 
Philip Martin | Subversion Committer
WANdisco // *Non-Stop Data*