search

vext01 / hgd

Hackathon Gunther Daemon :: Opensource Multi-User Jukebox
http://hgd.theunixzoo.co.uk
ISC License
22 stars 10 forks source link

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

Closed vext01 closed 13 years ago

vext01 commented 13 years ago

% /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

vext01 commented 13 years ago

note, i changed the subcommand name.

vext01 commented 13 years ago

is this done?

egelmex commented 13 years ago

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: https://github.com/vext01/hgd/issues/97#issuecomment-2310717

vext01 commented 13 years ago

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

vext01 commented 13 years ago

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:

vext01 commented 13 years ago

done