<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/1399/">https://reviewboard.asterisk.org/r/1399/</a>
</td>
</tr>
</table>
<br />
<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 Terry Wilson.</div>
<p style="color: grey;"><i>Updated Sept. 6, 2011, 6:25 p.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Changes</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;">Added option for autopauseunavail and updated CHANGES.</pre>
</td>
</tr>
</table>
<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;">Long ago, autopause used to also affect busy queue members. That behavior was changed. In some situations, it is conceivable that the old behavior might want to be retained. This patch adds the autopausebusy option to allow this behavior.
As an example:
If queue members are not allowed to make outbound calls, then the only calls that they receive will be from the Queue. The devices should be marked properly as IN USE when a call is in progress. But, if a queue member sets their phone to DND, it will respond as though it is busy even though it is not in use. Having the phone automatically paused in this case would make the phone then unavailable and settings like leave/joinwhenempty behave as expected. When the user is no longer on DND, they can unpause themselves.
NOTE: Asterisk currently will treat SIP responses 486, 600, and 603 as busy. Many phones send one of these responses for being on DND. Some send a 480 Temporarily Unavailable which Asterisk treats as CONGESTION. There are several different meanings that could be attached to a 480, so whether or not we should treat it as a BUSY instead is a topic for debate and could be addressed separately, but if we don't make it a BUSY condition then perhaps an "autopausecondition=busy|congestion option might be appropriate). See http://tools.ietf.org/html/draft-elwell-bliss-dnd-01#section-5.1 for some discussion of the DND issue.</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;">Set phone to DND, called queue with an unpaused member. Verified that the member became paused after first call attempt.</pre>
</td>
</tr>
</table>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="https://issues.asterisk.org/jira/browse/ASTERISK-16112">ASTERISK-16112</a>
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> (updated)</h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>/branches/10/CHANGES <span style="color: grey">(334572)</span></li>
<li>/branches/10/apps/app_queue.c <span style="color: grey">(334572)</span></li>
<li>/branches/10/configs/queues.conf.sample <span style="color: grey">(334572)</span></li>
</ul>
<p><a href="https://reviewboard.asterisk.org/r/1399/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>