MaddieM4 / EJTP-lib-python

Encrypted JSON Transport Protocol library for Python 2.6-3.3.
GNU Lesser General Public License v3.0
23 stars 10 forks source link

Add 'rm' subcommand to ejtp-identity ($25) #126

Closed MaddieM4 closed 11 years ago

MaddieM4 commented 11 years ago

While actually playing around with the script, I discovered I missed one obvious, important feature in the spec for #122: the ability to delete identities.

$ ejtp-identity list
mitzi@lackadaisy.com (rsa)
victor@lackadaisy.com (rsa)
atlas@lackadaisy.com (rsa)
$ ejtp-identity rm victor@lackadaisy.com
$ ejtp-identity list
mitzi@lackadaisy.com (rsa)
atlas@lackadaisy.com (rsa)

Should be able to take any number of arguments, one per identity to be deleted, and print a warning for identities that don't exist.

MaddieM4 commented 11 years ago

Bounty is $25.

http://www.freedomsponsors.org/core/issue/230/add-rm-subcommand-to-ejtp-identity


(Copied from acceptance criteria)

Add an 'rm' subcommand that takes a variable number of identity names, deletes each identity from the relevant cache files, and prints a warning for identities that are specified but don't exist in the cache.

iurisilvio commented 11 years ago

What happens if an identity exists in more than one file? Removed all identities with the same name or just the first one?

MaddieM4 commented 11 years ago

My current inclination: die with a warning that you would be deleting from the following files: <list of files>, and recommend either:

So if you know you want to delete Joe from all your sources, you can do ejtp-identity rm joe@blow.net -A.