diff options
author | Matthew Jordan <mjordan@digium.com> | 2014-05-28 22:54:12 +0000 |
---|---|---|
committer | Matthew Jordan <mjordan@digium.com> | 2014-05-28 22:54:12 +0000 |
commit | fb5690ce4b9ff896a56fe9b9eb9eeed1a2b5f028 (patch) | |
tree | 5fc5aa9c8099167cf81dc9cfdadb19ce3aa24743 /main/logger.c | |
parent | 812f33d22218817f32ce856969e8196475f6d5fd (diff) |
Logger/CLI/etc.: Fix some aesthetic issues; reduce chatty verbose messages
This patch addresses some aesthetic issues in Asterisk. These are all just
minor tweaks to improve the look of the CLI when used in a variety of
settings. Specifically:
* A number of chatty verbose messages were removed or demoted to DEBUG
messages. Verbose messages with a verbosity level of 5 or higher were -
if kept as verbose messages - demoted to level 4. Several messages
that were emitted at verbose level 3 were demoted to 4, as announcement
of dialplan applications being executed occur at level 3 (and so the
effects of those applications should generally be less).
* Some verbose messages that only appear when their respective 'debug'
options are enabled were bumped up to always be displayed.
* Prefix/timestamping of verbose messages were moved to the verboser
handlers. This was done to prevent duplication of prefixes when the
timestamp option (-T) is used with the CLI.
* Verbose magic is removed from messages before being emitted to
non-verboser handlers. This prevents the magic in multi-line verbose
messages (such as SIP debug traces or the output of DumpChan) from
being written to files.
* _Slightly_ better support for the "light background" option (-W) was
added. This includes using ast_term_quit in the output of XML
documentation help, as well as changing the "Asterisk Ready" prompt to
bright green on the default background (which stands a better chance of
being displayed properly than bright white).
Review: https://reviewboard.asterisk.org/r/3547/
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@414798 65c4cc65-6c06-0410-ace0-fbb531ad65f3
Diffstat (limited to 'main/logger.c')
-rw-r--r-- | main/logger.c | 59 |
1 files changed, 40 insertions, 19 deletions
diff --git a/main/logger.c b/main/logger.c index 2c0f3f4a3..16a044a6e 100644 --- a/main/logger.c +++ b/main/logger.c @@ -235,6 +235,7 @@ static const int colors[NUMLOGLEVELS] = { }; AST_THREADSTORAGE(verbose_buf); +AST_THREADSTORAGE(verbose_build_buf); #define VERBOSE_BUF_INIT_SIZE 256 AST_THREADSTORAGE(log_buf); @@ -1171,25 +1172,50 @@ static void ast_log_vsyslog(struct logmsg *msg) syslog(syslog_level, "%s", buf); } +static char *logger_strip_verbose_magic(const char *message, int level) +{ + char *p; + char *stripped_message = ast_strdup(message); + char magic = -(level + 1); + + if (!stripped_message) { + return NULL; + } + + do { + p = strchr(stripped_message, (char)magic); + if (p) { + *p++ = ' '; + } + } while (p && *p); + + return stripped_message; +} + /*! \brief Print a normal log message to the channels */ static void logger_print_normal(struct logmsg *logmsg) { struct logchannel *chan = NULL; char buf[BUFSIZ]; struct verb *v = NULL; + char *tmpmsg; int level = 0; if (logmsg->level == __LOG_VERBOSE) { - char *tmpmsg = ast_strdupa(logmsg->message + 1); - - level = VERBOSE_MAGIC2LEVEL(logmsg->message); /* Iterate through the list of verbosers and pass them the log message string */ AST_RWLIST_RDLOCK(&verbosers); AST_RWLIST_TRAVERSE(&verbosers, v, list) v->verboser(logmsg->message); AST_RWLIST_UNLOCK(&verbosers); - ast_string_field_set(logmsg, message, tmpmsg); + + level = VERBOSE_MAGIC2LEVEL(logmsg->message); + + tmpmsg = logger_strip_verbose_magic(logmsg->message, level); + if (tmpmsg) { + ast_string_field_set(logmsg, message, tmpmsg); + ast_free(tmpmsg); + } } AST_RWLIST_RDLOCK(&logchannels); @@ -1746,9 +1772,8 @@ void ast_log_backtrace(void) void __ast_verbose_ap(const char *file, int line, const char *func, int level, struct ast_callid *callid, const char *fmt, va_list ap) { const char *p; - struct ast_str *prefixed, *buf = NULL; + struct ast_str *prefixed, *buf; int res = 0; - const char *prefix = level >= 4 ? VERBOSE_PREFIX_4 : level == 3 ? VERBOSE_PREFIX_3 : level == 2 ? VERBOSE_PREFIX_2 : level == 1 ? VERBOSE_PREFIX_1 : ""; signed char magic = level > 9 ? -10 : -level - 1; /* 0 => -1, 1 => -2, etc. Can't pass NUL, as it is EOS-delimiter */ /* For compatibility with modules still calling ast_verbose() directly instead of using ast_verb() */ @@ -1767,18 +1792,18 @@ void __ast_verbose_ap(const char *file, int line, const char *func, int level, s } if (!(prefixed = ast_str_thread_get(&verbose_buf, VERBOSE_BUF_INIT_SIZE)) || - !(buf = ast_str_create(VERBOSE_BUF_INIT_SIZE))) { + !(buf = ast_str_thread_get(&verbose_build_buf, VERBOSE_BUF_INIT_SIZE))) { return; } res = ast_str_set_va(&buf, 0, fmt, ap); /* If the build failed then we can drop this allocated message */ if (res == AST_DYNSTR_BUILD_FAILED) { - ast_free(buf); return; } ast_str_reset(prefixed); + /* for every newline found in the buffer add verbose prefix data */ fmt = ast_str_buffer(buf); do { @@ -1787,22 +1812,12 @@ void __ast_verbose_ap(const char *file, int line, const char *func, int level, s } ++p; - if (ast_opt_timestamp) { - struct ast_tm tm; - char date[40]; - struct timeval now = ast_tvnow(); - ast_localtime(&now, &tm, NULL); - ast_strftime(date, sizeof(date), dateformat, &tm); - ast_str_append(&prefixed, 0, "%c[%s] %s", (char) magic, date, prefix); - } else { - ast_str_append(&prefixed, 0, "%c%s", (char) magic, prefix); - } + ast_str_append(&prefixed, 0, "%c", (char)magic); ast_str_append_substr(&prefixed, 0, fmt, p - fmt); fmt = p; } while (p && *p); ast_log_callid(__LOG_VERBOSE, file, line, func, callid, "%s", ast_str_buffer(prefixed)); - ast_free(buf); } void __ast_verbose(const char *file, int line, const char *func, int level, const char *fmt, ...) @@ -2121,3 +2136,9 @@ void ast_logger_unregister_level(const char *name) AST_RWLIST_UNLOCK(&logchannels); } } + +const char *ast_logger_get_dateformat(void) +{ + return dateformat; +} + |