<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Dec 7, 2016 at 8:34 AM, George Joseph <span dir="ltr"><<a href="mailto:gjoseph@digium.com" target="_blank">gjoseph@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 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>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_-2498727136940066965m_-7975469716360798568HOEnZb"><font color="#888888"><br></font></span></blockquote><div><br></div></span><div>Yes, it would. </div></div></div></div></blockquote><div><br></div></span><div>AMI versions bumped:</div><div>Branch 13: 3.0.0</div><div>Branch 14 and master: 4.0.0</div></div></div></div></blockquote><div><br></div><div><br></div><div>OK, We backtracked on this:</div><div><br></div><div>A new command was added PJSIPShowRegistrationInboundContactStatuses instead of modifying the old one.  No version bumps of any kind</div><div><br></div><div><br></div></div>
</div></div>