<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://reviewboard.asterisk.org/r/1599/">https://reviewboard.asterisk.org/r/1599/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On November 28th, 2011, 4:06 a.m., <b>Tzafrir Cohen</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<table width="100%" border="0" bgcolor="white" style="border: 1px solid #C0C0C0; border-collapse: collapse; margin: 2px padding: 2px;">
<thead>
<tr>
<th colspan="4" bgcolor="#F0F0F0" style="border-bottom: 1px solid #C0C0C0; font-size: 9pt; padding: 4px 8px; text-align: left;">
<a href="https://reviewboard.asterisk.org/r/1599/diff/5/?file=21910#file21910line97" style="color: black; font-weight: bold; text-decoration: underline;">/trunk/configs/logger.conf.sample</a>
<span style="font-weight: normal;">
(Diff revision 5)
</span>
</th>
</tr>
</thead>
<tbody style="background-color: #e4d9cb; padding: 4px 8px; text-align: center;">
<tr>
<td colspan="4"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>
</tr>
</tbody>
<tbody>
<tr>
<th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2"></font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>
<th bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">97</font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">; Messages with higher levels will be ignored. If verbose is specified at</pre></td>
</tr>
</tbody>
</table>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Minor issue: A bare 'verbose' used to mean verbose(5) (with the terminology of current changes). Now it means verbose(3). Maybe the default should still be 5 for backward compatibility?</pre>
</blockquote>
<p>On November 28th, 2011, 9:45 a.m., <b>Tilghman Lesher</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">No, it inherited whatever level was active when Asterisk was started. If you started Asterisk without ANY v's (or a verbose= setting in asterisk.conf), then it was 0. And worse, if you changed the verbosity level during runtime, you would affect what would go to the log.</pre>
</blockquote>
</blockquote>
<pre style="margin-left: 1em; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Most people, when they start Asterisk for ad-hoc testing, simply hold down the v key for a brief period. That rarely means that verbosity is consistent, runtime-to-runtime. Could be 5 or 7 or 16 or 22. We could inherit from whatever option_verbose is set to at the time. My issue with that, though, is that the verbosity level can be changed during runtime and a 'logger reload' would thus change the logger verbosity level, which
most people would consider unexpected behavior.
Thus, I think 3 is an acceptable default.</pre>
<br />
<p>- Tilghman</p>
<br />
<p>On November 28th, 2011, 10:15 a.m., Tilghman Lesher wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://reviewboard.asterisk.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Asterisk Developers.</div>
<div>By Tilghman Lesher.</div>
<p style="color: grey;"><i>Updated Nov. 28, 2011, 10:15 a.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">One problem that many admins have today is that to have maximum verbosity going to the logs, for later debugging of trouble calls, the console becomes virtually unusable, because the verbosity level is global. This patch attempts to solve that problem, allowing each verbose recipient to have its own filter as to what it wants to receive.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Verified logging with files, consoles with different verbosity levels, etc.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>/trunk/addons/chan_ooh323.c <span style="color: grey">(346290)</span></li>
<li>/trunk/addons/res_config_mysql.c <span style="color: grey">(346290)</span></li>
<li>/trunk/apps/app_rpt.c <span style="color: grey">(346290)</span></li>
<li>/trunk/apps/app_verbose.c <span style="color: grey">(346290)</span></li>
<li>/trunk/apps/app_voicemail.c <span style="color: grey">(346290)</span></li>
<li>/trunk/channels/chan_sip.c <span style="color: grey">(346290)</span></li>
<li>/trunk/channels/chan_skinny.c <span style="color: grey">(346290)</span></li>
<li>/trunk/channels/chan_usbradio.c <span style="color: grey">(346290)</span></li>
<li>/trunk/codecs/codec_dahdi.c <span style="color: grey">(346290)</span></li>
<li>/trunk/configs/logger.conf.sample <span style="color: grey">(346290)</span></li>
<li>/trunk/include/asterisk/logger.h <span style="color: grey">(346290)</span></li>
<li>/trunk/main/asterisk.c <span style="color: grey">(346290)</span></li>
<li>/trunk/main/bridging.c <span style="color: grey">(346290)</span></li>
<li>/trunk/main/cli.c <span style="color: grey">(346290)</span></li>
<li>/trunk/main/dial.c <span style="color: grey">(346290)</span></li>
<li>/trunk/main/logger.c <span style="color: grey">(346290)</span></li>
<li>/trunk/main/say.c <span style="color: grey">(346290)</span></li>
<li>/trunk/res/res_clialiases.c <span style="color: grey">(346290)</span></li>
<li>/trunk/res/res_fax.c <span style="color: grey">(346290)</span></li>
<li>/trunk/res/res_jabber.c <span style="color: grey">(346290)</span></li>
<li>/trunk/res/res_musiconhold.c <span style="color: grey">(346290)</span></li>
</ul>
<p><a href="https://reviewboard.asterisk.org/r/1599/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>