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

Re: [PATCH] Invoke 'store password' prompt for all commands requiring authentication

From: C. Michael Pilato <cmpilato_at_collab.net>
Date: Wed, 07 Oct 2009 08:54:33 -0400

Kannan wrote:
> Stefan Sperling wrote:
>
>> Does this patch fix this case, too?
>
> It doesn't as of now, also for other svn prop* commands. Trying to fix
> them too.

Looking through the ra_neon code, I'm not sure I could detect either rhyme
or reason for when the maybe_store_auth_info() help was being called. Looks
like we just scattered those calls all over the place as we detected the
need for them.

ra_serf appears to take a more sane approach, remembering in a very
low-level request handler helper function when it had to re-attempt a
request (any request!) on account of insufficient privileges, and storing
auth creds if the second attempt succeeded.

-- 
C. Michael Pilato <cmpilato_at_collab.net>
CollabNet   <>   www.collab.net   <>   Distributed Development On Demand
------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=2404498

Received on 2009-10-07 14:54:48 CEST

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.