[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [PATCH] Handle failed authentication in ra_serf

From: Justin Erenkrantz <justin_at_erenkrantz.com>
Date: 2007-06-24 00:43:08 CEST

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

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.