[Freeipa-devel] [PATCH 0072] Provide ipa-client-advise tool

[Tomas Babej]

On Wednesday 26 of June 2013 10:12:48 Petr Spacek wrote:

[snip]

> 
> Appropriate error handling = Return 'Permission denied' if particular 
> operation requires higher privileges.
> 
> IMHO 'cryptic' error message is bad in any case, so the right way how to fix 
> 'cryptic' error messages is to fix the places where errors are thrown.
> 
> I don't think that additional checks in 'advisor' to hide 'cryptic' errors 
are 
> the right approach.
> 
> -- 
> Petr^2 Spacek

To wrap-up, after an offline discussion Petr:

We came to an conclusion that since 'require_root' attribute is optional (Petr 
was not aware of that, and that fact was what caused his concern), with 
default value False, the attribute should not pose any additional burden for
the plugin developer.

Tomas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20130711/8a1fd660/attachment.htm>