[email protected] 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*
