log_level and log_type should be ChoiceOptions
|Reported by:||Ryan J Ollos||Owned by:||Ryan J Ollos|
If we change
TracError will be raised when an invalid value is used.
[logging] log_level = DEBUG log_type = stderror
Error TracError: [logging] log_type: expected one of ("file", "none", "stderr", "syslog", "winlog"), got u'stderror'
This could be useful, but will be challenged by the presence of aliases:
- ALL is an alias for
- WARN is documented as an alias for
WARNING, and it appears that the default is WARNING for any unrecognized value.
Possible resolutions are:
- Eliminate some aliases and replace them in trac.ini in a upgrade step.
ChoiceOptionto support aliases.
I probably won't take any action on this for a while, but prepared some proposed changes in log:rjollos.git:t11573 before noticing these issues, and wanted to capture them in a ticket.
Change History (13)
comment:11 by , 5 years ago
|Release Notes:||modified (diff)|
|Status:||assigned → closed|