--change-password functionality #13

Merged
merged 6 commits into from Dec 5, 2012

Conversation

Projects
None yet
3 participants
Contributor

thenoviceoof commented Jul 6, 2012

Batch changes the password applied to the keys records, if the old password was compromised or you merely wish to change it, and be able to keep using the sqlite database.

thenoviceoof added some commits Jul 1, 2012

Added --change-password functionality
When run, it will prompt for 2 passwords: the first decrypts the
keyboard input, and then reencrypts it with the new password.
The utility requires a lock on the database, so it requires selfspy
to not be running while it changes the passwords. Also, it does not
run the logger after automatically, because being explicit is
better.
Collaborator

gurgeh commented Jul 16, 2012

I don't think it is a good idea to change the prints to py3k style. It is not backwards compatible with 2.x. See for example the difference between:
print 5,
print 7 #outputs 5 7
and
print(5,)
print(7) #outputs (5,)\n7

Since 2.x is what the rest of the code supports, changing the prints seems odd.

Collaborator

gurgeh commented Jul 16, 2012

Otherwise the feature looks good! Care to remove the py3k commit from the pull request, so I can merge it automatically?

Contributor

thenoviceoof commented Jul 17, 2012

Done, had py3k on the brain at the time

Contributor

ljos commented Dec 4, 2012

@gurgeh This has been here for a while. Maybe we should look into merging it if the feature is not there?

@ljos ljos referenced this pull request Dec 5, 2012

Merged

Merge thenoviceoof to master #36

@gurgeh gurgeh merged commit 7b2409c into selfspy:master Dec 5, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment