On 6/23/07, Lieven Govaerts <lgo@mobsol.be> wrote:
>
> Attached is a small patch that handles failed authentication in ra_serf.
> With current trunk when you provide wrong credentials for four times svn
> will get in an endless loop.
>
> I couldn't find an error code for 'Authentication failed', it seems to
> me that other ra layers are incorrectly using Authori*z*ation failed as
> error message.
>
> Unless anyone objects, I'll commit this tomorrow.
Looks good - one minor suggestion: tweak handle_auth to return
svn_error_t and return Authentication failed error inside handle_auth
(and still do the pending_error set in handle_response). For the
abort() cases we talked about on IRC, we can return more informative
error descriptions as well... -- justin
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sun Jun 24 00:42:52 2007