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

RE: UI Proposal for Code Collaborator settings

From: Friedrich Brunzema <brunzefb_at_yahoo.com>
Date: Fri, 7 Jun 2013 16:39:02 -0700 (PDT)

More info - formum access via mail does not seem to work...

On Fri, Jun 7, 2013 at 9:22 PM, Stefan K√ľng <tortoisesvn_at_gmail.com> wrote:

    On 07.06.2013 21:17, friedrich brunzema wrote:

        I could also detect and show this settings pg only when installed.
        Problem with the
        Ini file is how to set encrypted passwords.
        Even on multiple projects, the Id and pw are not likely different.
        Would you more comfortable with proceeding like this?

    Not sure what problem you see storing encrypted passwords in an ini file - it's not much different than storing them in the registry as you do now?

    If you don't mind, I'll revert your change for now because it's not finished yet and I'd like to create the 1.8 branch.

Ok, I've reverted your changes for now.
And I've added crypt methods to the string utilities so you don't have to misuse the creds class to encrypt your data.
I've also moved your class to separate files.

Now, as for where to store and get the required information:

PathToCollabGui: is that really necessary? I mean you could just request that every user put that path to the PATH environment variable and you're done.
RepoUrl: is that different from the repo root url? If yes is there maybe another way to get the url?

CollabUser/Password, SvnUser/Password: why not just show a dialog asking for that information the first time CodeCollaboratorInfo::GetCommandLineArguments is called and those strings are empty? Then you can store that information wherever you want.

Enabling the whole feature would be done using a project property so projects who don't use it won't even see it.


  oo  // \\      "De Chelonian Mobile"
 (_,\/ \_/ \     TortoiseSVN
   \ \_/_\_/>    The coolest Interface to (Sub)Version Control
   /_/   \_\     http://tortoisesvn.net
To unsubscribe from this discussion, e-mail: [dev-unsubscribe_at_tortoisesvn.tigris.org].
Received on 2013-06-08 01:39:05 CEST

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

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