<div dir="ltr"><div>Hi,</div><div><br></div>I submitted that change, as we have need for not the 'no' and the 'never' cases on different devices/trunks etc, and before the patch they were almost the same.<br><br>I completely agree with the above suggestion from Kevin. I always set "progressinband" manually for all of my device definitions, so had never realised that the patch was changing the normal/default behaviour.<div><br></div><div>Regards,</div><div>Steve</div></div><br><div class="gmail_quote">On Wed, 8 Apr 2015 at 18:00 Kevin Harwell <<a href="mailto:kharwell@digium.com">kharwell@digium.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><div>Greetings,<br><br></div>A few months ago an issue [1] was reported that when the "progressinband" option found in chan_sip was set to "never" it was not working correctly in some scenarios. A fix for the issue was committed [2]. However, the fix had a side effect of changing the scope of "never" somewhat and since "never" is the default this can potentially cause some confusion or unexpected behavior when upgrading.<br><br></div>It's believed that changing the default from "never" to "no" would be more appropriate for most users as then "progressinband" would default to working similar to how it was before.<br><br></div>Thoughts on this? Leave it defaulting to "never"? Changing it to "no" would be fine?<br><div><div><br>[1] <a href="https://issues.asterisk.org/jira/browse/ASTERISK-23972" target="_blank">https://issues.asterisk.org/jira/browse/ASTERISK-23972</a><br>[2] <a href="https://reviewboard.asterisk.org/r/3700" target="_blank">https://reviewboard.asterisk.org/r/3700</a><br clear="all"><div><div><br>-- <br><div><div dir="ltr"><pre style="padding:2px;border:1px solid rgb(114,99,77);background-color:rgb(238,238,238);color:rgb(0,0,0);overflow:auto">Kevin Harwell
Digium, Inc. | Software Developer
445 Jan Davis Drive NW - Huntsville, AL 35806 - USA
Check us out at: <a href="http://digium.com" target="_blank">http://digium.com</a> & <a href="http://asterisk.org" target="_blank">http://asterisk.org</a></pre></div></div>
</div></div></div></div></div>
--<br>
______________________________<u></u>______________________________<u></u>_________<br>
-- Bandwidth and Colocation Provided by <a href="http://www.api-digital.com" target="_blank">http://www.api-digital.com</a> --<br>
<br>
asterisk-dev mailing list<br>
To UNSUBSCRIBE or update options visit:<br>
   <a href="http://lists.digium.com/mailman/listinfo/asterisk-dev" target="_blank">http://lists.digium.com/<u></u>mailman/listinfo/asterisk-dev</a></blockquote></div>