[Freeipa-devel] [PATCH] add requires_root option to Command
Simo Sorce
ssorce at redhat.com
Tue Apr 21 21:10:46 UTC 2009
On Tue, 2009-04-21 at 10:24 -0400, Rob Crittenden wrote:
> Some commands will require that the local user have root permissions.
> I'm not 100% sure this is the right place to put it but it at least
> starts the conversation.
Speaking just in general terms I don't like doings things like:
if uid == 0 fail;
I think that we should gracefully catch whatever exception is thrown up
(access denied or whatever) and then return an error.
Some times this is not possible, and I haven't looked at what's around
that patch, so this may be the right way in this case.
Simo.
More information about the Freeipa-devel
mailing list