Justin Erenkrantz wrote:
> 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
Yep, good point. r25519 already removes some of the abort() calls.
Lieven
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sun Jun 24 01:14:31 2007