<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/2959/">https://reviewboard.asterisk.org/r/2959/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On October 24th, 2013, 10:50 p.m. UTC, <b>Paul Belanger</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;">My only comment is about the naming conventions, we should try and keep AMI events in sync. Right now we have FooBegin / FooEnd, and FooStart / FooStop, and Foo / FooComplete. I'd like us see about pick one format and sticking with it when possible.
I believe Begin / End has been the most recent addition, do we want to default to that?</pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">There is no proposed convention for that in AMI 1.4. I really don't want to go through and refactor all of those events again, particularly since that specification was put up for review nearly a year ago.
That being said, the Begin/End and Start/Stop nomenclature could be considered to be inconsistent; in general, people have gone with what they thought made sense based on the semantics the event was conveying. If someone want to propose a consistent scheme for that I'd be fine with that, but I'd rather see that on the -dev list and - once someone has made a decision - a patch provided that does the change.
As far as Foo/FooComplete is concerned, however, those are used to indicate when a list of events has finished that was initiated by an action. Those are not inconsistent with the others, and changing them to something else feels incredibly nit picky.
As far as the substatus field goes, the specification for AMI went with explicit event names to indicate the beginning and ending of a pair of events. That approach was chosen as it allows libraries to key off of the begin or end events if that is the only event that they care about; this reduces the amount of work implementers have to do as they don't have to inspect a sub type field to determine whether or not they care about the event.
</pre>
<br />
<p>- Matt</p>
<br />
<p>On October 24th, 2013, 5:43 p.m. UTC, Kevin Harwell wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://reviewboard.asterisk.org/static/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 Kevin Harwell.</div>
<p style="color: grey;"><i>Updated Oct. 24, 2013, 5:43 p.m.</i></p>
<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-22609">ASTERISK-22609</a>
</div>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
Asterisk
</div>
<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;">Created the following AMI commands and corresponding events for res_pjsip:
PJSIPShowEndpoints - Provides a listing of all pjsip endpoints and a few select attributes on each.
Events:
EndpointList - for each endpoint a few attributes
EndpointlistComplete - after all endpoints have been listed
PJSIPShowEndpoint - Provides a detail list of attributes for a specified endpoint
Events:
EndpointDetail - attributes on an endpoint
AorDetail - raised for each AOR on an endpoint
AuthDetail - raised for each associated inbound and outbound auth
TransportDetail - transport attributes
IdentifyDetail - attributes for the identify object associated with the endpoint
EndpointDetailComplete - last event raised after all detail events
PJSIPShowRegistrationsInbound - Provides a detail listing of all inbound registrations
Events:
InboundRegistrationDetail - inbound registration attributes for each registration
InboundRegistrationDetailComplete - raised after all detail records have been listed
PJSIPShowRegistrationsOutbound - Provides a detail listing of all outbound registrations
Events:
OutboundRegistrationDetail - outbound registration attributes for each registration
OutboundRegistrationDetailComplete - raised after all detail records have been listed
PJSIPShowSubscriptions - A detail listing of all subscriptions and their attributes
Events:
SubscriptionDetail - on each subscription detailed attributes
SubscriptionDetailComplete - raised after all detail records have been listed</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;">Ran a few manual tests over AMI and observed the output to make sure things look correct. Also wrote some testsuite tests: https://reviewboard.asterisk.org/r/2958/</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>branches/12/include/asterisk/acl.h <span style="color: grey">(401659)</span></li>
<li>branches/12/include/asterisk/manager.h <span style="color: grey">(401659)</span></li>
<li>branches/12/include/asterisk/res_pjsip.h <span style="color: grey">(401659)</span></li>
<li>branches/12/include/asterisk/res_pjsip_pubsub.h <span style="color: grey">(401659)</span></li>
<li>branches/12/include/asterisk/sorcery.h <span style="color: grey">(401659)</span></li>
<li>branches/12/include/asterisk/strings.h <span style="color: grey">(401659)</span></li>
<li>branches/12/main/acl.c <span style="color: grey">(401659)</span></li>
<li>branches/12/main/manager.c <span style="color: grey">(401659)</span></li>
<li>branches/12/main/sorcery.c <span style="color: grey">(401659)</span></li>
<li>branches/12/main/utils.c <span style="color: grey">(401659)</span></li>
<li>branches/12/res/res_pjsip.c <span style="color: grey">(401659)</span></li>
<li>branches/12/res/res_pjsip.exports.in <span style="color: grey">(401659)</span></li>
<li>branches/12/res/res_pjsip/config_auth.c <span style="color: grey">(401659)</span></li>
<li>branches/12/res/res_pjsip/config_transport.c <span style="color: grey">(401659)</span></li>
<li>branches/12/res/res_pjsip/include/res_pjsip_private.h <span style="color: grey">(401659)</span></li>
<li>branches/12/res/res_pjsip/location.c <span style="color: grey">(401659)</span></li>
<li>branches/12/res/res_pjsip/pjsip_configuration.c <span style="color: grey">(401659)</span></li>
<li>branches/12/res/res_pjsip_endpoint_identifier_ip.c <span style="color: grey">(401659)</span></li>
<li>branches/12/res/res_pjsip_exten_state.c <span style="color: grey">(401659)</span></li>
<li>branches/12/res/res_pjsip_mwi.c <span style="color: grey">(401659)</span></li>
<li>branches/12/res/res_pjsip_outbound_registration.c <span style="color: grey">(401659)</span></li>
<li>branches/12/res/res_pjsip_pubsub.c <span style="color: grey">(401659)</span></li>
<li>branches/12/res/res_pjsip_registrar.c <span style="color: grey">(401659)</span></li>
</ul>
<p><a href="https://reviewboard.asterisk.org/r/2959/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>