<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Dec 7, 2016 at 8:16 AM, Matthew Jordan <span dir="ltr"><<a href="mailto:mjordan@digium.com" target="_blank">mjordan@digium.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><span class="">On Tue, Dec 6, 2016 at 4:27 PM, Matt Fredrickson <span dir="ltr"><<a href="mailto:creslin@digium.com" target="_blank">creslin@digium.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>On Tue, Dec 6, 2016 at 12:30 PM, Joshua Colp <<a href="mailto:jcolp@digium.com" target="_blank">jcolp@digium.com</a>> wrote:<br>
> On Tue, Dec 6, 2016, at 10:43 AM, George Joseph wrote:<br>
>> We just discovered that the PJSIPShowRegistrationsInbound command really<br>
>> only dumps all aors regardless of whether there's an inbound registration<br>
>> associated with it or not.  Fixing this would mean a change to what this<br>
>> command returns so I'm trying to get some feedback.  There are 2 solution<br>
>> alternatives...<br>
>><br>
>> 1.  We could replace the current InboundRegistrationDetail event (which<br>
>> isn't even registered) with a ContactStatusDetail event.  Obviously this<br>
>> is<br>
>> a change for anyone who uses this command.<br>
>><br>
>> 2.  We could send a ContactStatusDetail event along with the existing<br>
>> InboundRegistrationDetail event.  This would double the number of events<br>
>> sent and therefore increase the total data sent.<br>
>><br>
>> Honestly I'm not sure how this command was ever useful to anybody so I'm<br>
>> leaning towards option 1 but we need feedback.<br>
>><br>
>> This would be a change to 13, 14 and master.<br>
><br>
> Since it was not really useful I'm okay with 1.<br>
<br>
</span>Just as something to consider:<br>
<br>
If we make breaking changes to AMI, this would require a major number<br>
version bump, right?<br>
<span class="m_-7975469716360798568HOEnZb"><font color="#888888"><br></font></span></blockquote><div><br></div></span><div>Yes, it would. </div></div></div></div></blockquote><div><br></div><div>AMI versions bumped:</div><div>Branch 13: 3.0.0</div><div>Branch 14 and master: 4.0.0</div><div> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div><br></div>-- <br><div class="m_-7975469716360798568gmail_signature" data-smartmail="gmail_signature">Matthew Jordan<br>Digium, Inc. | CTO<span class=""><br>445 Jan Davis Drive NW - Huntsville, AL 35806 - USA<br></span>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></div>
</div></div>
<br>--<br>
______________________________<wbr>______________________________<wbr>_________<br>
-- Bandwidth and Colocation Provided by <a href="http://www.api-digital.com" rel="noreferrer" 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" rel="noreferrer" target="_blank">http://lists.digium.com/<wbr>mailman/listinfo/asterisk-dev</a><br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><span style="font-size:12.8px">George Joseph</span><br style="font-size:12.8px"><span style="font-size:12.8px">Digium, Inc. | Software Developer</span><span style="font-size:12.8px"><br>445 Jan Davis Drive NW - Huntsville, AL 35806 - US<br></span><span style="font-size:12.8px">Check us out at: </span><a href="http://www.digium.com/" rel="noreferrer" style="color:rgb(17,85,204);font-size:12.8px" target="_blank">www.digium.com</a><span style="font-size:12.8px"> & </span><a href="http://www.asterisk.org/" rel="noreferrer" style="color:rgb(17,85,204);font-size:12.8px" target="_blank">www.asterisk.org</a><br><div><br></div></div></div>
</div></div>