Command line edition enhancements

[dr4Ke]

Hello everybody.
I'm a happy user of the Firefox edition.
passwordmaker is not available for Google Chrome and i'm testing it now.
So, while waiting for a new edition for this software, i decided to try the command line edition (c++).

I've done some little work on the c++ command line edition.
Here are my changes :
leet.cpp :

• FIX: don't compile on Linux (added '#include <cstring>' to fix)
• FIX: warnings during compilation about (changed static char* to static const char*)

reading settings :

• parse settings from args
• THEN search account in the settings file
• IF found, take only settings that were not given by a command line option
• IF NOT found, use default account settings (if not cmd line option)

other changes :

• Added a message if the given account was not found in the settings file
• Print ********* instead of the master password in verbose mode
• Show settings file name in verbose mode
• Show trimzeroes setting in verbose mode
• Show account name in verbose mode (including the default account name)

Attached a diff file.

Would you mind take a look at my changes ?

Thank you.
dr4Ke.

[Miquel 'Fire' Burns]

I'll review when I have the chance.

[Miquel 'Fire' Burns]

I reviewed it, and noticed that you messed up the default settings on the help screen.

How will someone know the default values when they view the help? I'll check on that later.

[dr4Ke]

Ouch ! I didn't look much at the help screen.

I'll look at that tomorrow.

[dr4Ke]

I checked on that. The default settings are the same as before :

My version :

$ ./passwordmaker --verbose
Master Password: 
Master Password: ********
Using settings file: passwordmaker.rdf
Using Account: 
Use l33t: none
l33t level: 0
Hash Algorithm: MD5
Triming trailing zeroes: yes
URL: 
Generated Password Length: 8
Username: 
Modifier: 
Characters: ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789`~!@#$%^&*()_-+={}|[]\:";'<>?,./
Generated Password Prefix: 
Generated Password Suffix: 
D4a8=qpD

Trunk version :

./passwordmaker --verbose
Master Password: 
Master Password: youpi
Use l33t: none
l33t level: 0
Hash Algorithm: MD5
URL: 
Generated Password Length: 8
Username: 
Modifier: 
Characters: ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789`~!@#$%^&*()_-+={}|[]\:";'<>?,./
Generated Password Prefix: 
Generated Password Suffix: 
D4a8=qpD

The change is about separate reading settings on the command line and in the settings file. It's why I changed default values on the tclap (to see the difference between the default value and a user's option on the command line). The real defaults values (used to process the password) are set when the pwmdefaults object is created and are copied in the settings object by the function getSettings only if there was the corresponding option was not given on the command line.

I hope this will clarify what I changed.

Hope that my english is readable.

Anyway, thanks for checking my changes. I'll be glad to contribute to the project.
dr4Ke.

[Miquel 'Fire' Burns]

Run ./passwordmaker -h as see what I mean by the default settings.

[dr4Ke]

Run ./passwordmaker -h as see what I mean by the default settings.

I did.
I don't see any change between my version and the original version in trunk (except the description string shown at the end of the help screen).