URL:
https://github.com/freeipa/freeipa/pull/3158
Author: rcritten
Title: #3158: admintool: don't display log file on errors unless logging is setup
Action: opened
PR body:
"""
The admintool will display the message when something goes wrong:
See %s for more information" % self.log_file_name
This is handy except when finally logging setup is not done
yet so the log file doesn't actually get written to.
This can happen if validation catches and raises an exception.
The workaround is to save off and remove the log_file_name value
before calling validation, then restore it if successful. This will
suppress the above error message.
Fixes:
https://pagure.io/freeipa/issue/7952
Signed-off-by: Rob Crittenden <rcritten(a)redhat.com>
"""
To pull the PR as Git branch:
git remote add ghfreeipa
https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/3158/head:pr3158
git checkout pr3158