diff options
author | John Dennis <jdennis@redhat.com> | 2011-11-30 11:08:42 -0500 |
---|---|---|
committer | Martin Kosek <mkosek@redhat.com> | 2011-12-01 08:34:02 +0100 |
commit | 167813f34350466be569be26f4faa8b9a05e5920 (patch) | |
tree | e9df21a77042214ff5416c6e475d61378185e296 /ipa-client | |
parent | 27931dcb29ebeb15aae4d17d745252ba75f69bae (diff) | |
download | freeipa-167813f34350466be569be26f4faa8b9a05e5920.tar.gz freeipa-167813f34350466be569be26f4faa8b9a05e5920.tar.xz freeipa-167813f34350466be569be26f4faa8b9a05e5920.zip |
Restore default log level in server to INFO
The default log level for server messages captured by httpd's
error_log historically was INFO. The log_manager patch had it set to
ERROR, this patch resets it back to INFO.
Although it would have been trival to set the default_level to INFO in
IPALogManager.configure_from_env() that is not logically the correct
place. It would be much better if the default_level can be reset by
simply assigning it to the log_mgr. To accomplish that
LogManager.default_level was converted to a property with a getter and
setter. The setter runs LogManager.apply_configuratin() after the
default_level is modified. LogManager.set_default_level() was also
added to allow simultaneously updating the configure_state.
While testing some minor problems were observed and also fixed:
* Removed some print statement which had been left in by mistake
* Removed the ability to set the handler level in the config file
because of chicken-and-egg issues of when handlers get created.
The Env config file format is too inflexible to support detailed
logging configuration. If the Env config format is ever made more
flexible we can come back and add this back in. The handler config
setting in Env had never been used and never worked so there is no
issue in removing it.
Diffstat (limited to 'ipa-client')
0 files changed, 0 insertions, 0 deletions