[Asterisk-code-review] res pjsip registrar: Change RegistrationsInbound to send Co... (asterisk[master])

George Joseph asteriskteam at digium.com
Wed Dec 7 09:32:30 CST 2016


Hello Anonymous Coward #1000019, Joshua Colp,

I'd like you to reexamine a change.  Please visit

    https://gerrit.asterisk.org/4563

to look at the new patch set (#3).

Change subject: res_pjsip_registrar:  Change RegistrationsInbound to send ContactStatus
......................................................................

res_pjsip_registrar:  Change RegistrationsInbound to send ContactStatus

The PJSIPShowRegistrationsInbound AMI command was just dumping out
all AORs which was pretty useless and resource heavy since it had
to get all endpoints, then all aors for each endpoint, then all
contacts for each aor.

Now we send ContactStatusDetail events which meets the intended
purpose of the command and significantly reduces the overhead.
Also, some additional fields that were added to Contact since
the original creation of the ContactStatusDetail event have been
added to the end of the event.

AMI version bumped to 4.0.0

ASTERISK-26644 #close

Change-Id: I2468f9d8a9d703106b530cccec20ff3874211bdb
---
M CHANGES
M include/asterisk/manager.h
M include/asterisk/res_pjsip.h
M res/res_pjsip.c
M res/res_pjsip/pjsip_options.c
M res/res_pjsip_registrar.c
6 files changed, 84 insertions(+), 65 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/63/4563/3
-- 
To view, visit https://gerrit.asterisk.org/4563
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I2468f9d8a9d703106b530cccec20ff3874211bdb
Gerrit-PatchSet: 3
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>



More information about the asterisk-code-review mailing list