vext01/hgd

Should not be possible to double make/revoke admin a user

Closed this issue · 6 comments

% /home/edd/source/hgd/hgd-admin user-mk-admin edd
% /home/edd/source/hgd/hgd-admin user-mk-admin edd
% /home/edd/source/hgd/hgd-admin user-mk-admin edd
% /home/edd/source/hgd/hgd-admin user-mk-admin edd
% /home/edd/source/hgd/hgd-admin user-mk-admin edd
% /home/edd/source/hgd/hgd-admin user-mk-admin edd
% /home/edd/source/hgd/hgd-admin user-mk-admin edd
% /home/edd/source/hgd/hgd-admin user-mk-admin edd

note, i changed the subcommand name.

is this done?

Test make admin, if your happy will implement same for rm admin.

On Thu, Oct 6, 2011 at 3:54 PM, Edd Barrett
reply@reply.github.com
wrote:

is this done?

Reply to this email directly or view it on GitHub:
#97 (comment)

This needs more work:

unsafemist% ./hgd-admin user-mkadmin edd
User edd perms not changed. Probably already an admin.
unsafemist% ./hgd-admin user-noadmin edd
unsafemist% ./hgd-admin user-noadmin edd
unsafemist% ./hgd-admin user-noadmin edd
unsafemist% ./hgd-admin user-noadmin edd
unsafemist% ./hgd-admin user-noadmin edd
unsafemist% ./hgd-admin user-noadmin edd

same remotely:

unsafemist% ./hgd-admin user-mkadmin edd
unsafemist% ./hgdc user-mkadmin j
Password for edd@127.0.0.1:
unsafemist% ./hgdc user-mkadmin j
Password for edd@127.0.0.1:
unsafemist% ./hgdc user-mkadmin j
Password for edd@127.0.0.1:
unsafemist% ./hgdc user-noadmin j
Password for edd@127.0.0.1:
unsafemist% ./hgdc user-noadmin j
Password for edd@127.0.0.1:

done