[asterisk-bugs] [Asterisk 0017222]: [patch] [regression] Colourized logging option request as an option

[Asterisk Bug Tracker]

The following issue requires your FEEDBACK. 
====================================================================== 
https://issues.asterisk.org/view.php?id=17222 
====================================================================== 
Reported By:                voxter
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   17222
Category:                   Core/General
Reproducibility:            have not tried
Severity:                   feature
Priority:                   normal
Status:                     feedback
Asterisk Version:           1.6.2.7-rc2 
JIRA:                        
Regression:                 Yes 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Request Review:              
====================================================================== 
Date Submitted:             2010-04-21 14:03 CDT
Last Modified:              2010-04-21 15:09 CDT
====================================================================== 
Summary:                    [patch] [regression] Colourized logging option
request as an option
Description: 
I noticed that recently there was a patch submitted to remove colorized
escape sequences from log files,
https://issues.asterisk.org/view.php?id=16786

The decision was made to not offer this as an option, however, I find it
incredibly useful when debugging certain features to retain color codes in
the log files to read it easier.  This being suddenly gone, could be
considered a regression.

On IRC, Naikrovek mentioned that a recent discovery was made correlating
writing color codes to log files with CPU spikes.  Of course, if this is
infact the case, it should not be added as an option until the CPU spike is
solved (if it is possible to be solved, of course)

But, on the grounds that it was once an available feature, that I welcomed
and took advantage of for debugging purposes, I would like to see it
re-introduced as a configurable option.
======================================================================
Relationships       ID      Summary
----------------------------------------------------------------------
related to          0016786 [patch] Remove coloring escape sequence...
====================================================================== 

---------------------------------------------------------------------- 
 (0120724) tilghman (administrator) - 2010-04-21 15:09
 https://issues.asterisk.org/view.php?id=17222#c120724 
---------------------------------------------------------------------- 
I really don't see this as a regression, more as the reintroduction of a
bug.  Terminal codes should never have been going to the logfiles in the
first place.  There are certainly logfile readers that introduce color
codes for ease of reading, and I encourage the reporter to go down that
route, instead of adding this option to Asterisk. 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2010-04-21 15:09 tilghman       Note Added: 0120724                          
2010-04-21 15:09 tilghman       Status                   ready for testing =>
feedback
======================================================================