On 10/27/2015 04:24 PM, Tomas Babej wrote:
this couple of patches harden the adtrust installer.
Details in the commit messages.
in the first patch you forgot to instantiate the caught exception in the
+ except Exception:
+ root_logger.debug("Exception occured during SID generation:
You should use 'except Exception as e:'.
I'm also not quite sure that it is enough to log the error at debug level.
If the sidgen task somehow fails, isn't it something which should
interest the user and deserve at least warning-level message?
Manage your subscription for the Freeipa-devel mailing list:
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code