[asterisk-dev] [Code Review] 2523: Pimp my SIP: extension state notifications

Kevin Harwell reviewboard at asterisk.org
Thu May 23 18:00:05 CDT 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/2523/
-----------------------------------------------------------

(Updated May 23, 2013, 11 p.m.)


Review request for Asterisk Developers.


Changes
-------

Updated per review findings.  Removed horrible module registration work around.


Bugs: ASTERISK-21441 and ASTERISK-21442
    https://issues.asterisk.org/jira/browse/ASTERISK-21441
    https://issues.asterisk.org/jira/browse/ASTERISK-21442


Repository: Asterisk


Description
-------

Adds support for extension state notifications in the new SIP channel driver.  A simple API was developed so that specific extension state providers can attach and create the relevant NOTIFY request body, according to their registered type, for the outgoing request.  The framework maintains the list of providers and looks up the appropriate provider upon a state change for an associated registered subscription.  For instance, a provider can be registered for a particular event/media type tuple (ex: presence and application/xml+pidf).  When a subscription comes in for a given event/media type it is the responsibility of the registered provider to create that NOTIFY body.

Unfortunately pjsip only allows an event and its associated media types to be registered all at once, so providers are not in themselves modules, but in order to maintain some division and to make it easier to move into a module later providers can and should reside in separate files (found in res/res_sip_providers/).

Included in this review is also an implementation of providers for the following types: event=presence; media_type=application/pidf+xml, application/xml+pidf, and application/cpim+pidf+xml.


Diffs (updated)
-----

  team/group/pimp_my_sip/include/asterisk/res_sip_exten_state.h PRE-CREATION 
  team/group/pimp_my_sip/include/asterisk/res_sip_pubsub.h 389657 
  team/group/pimp_my_sip/res/res_sip_exten_state.c PRE-CREATION 
  team/group/pimp_my_sip/res/res_sip_exten_state.exports.in PRE-CREATION 
  team/group/pimp_my_sip/res/res_sip_pidf.c PRE-CREATION 
  team/group/pimp_my_sip/res/res_sip_pubsub.c 389657 
  team/group/pimp_my_sip/res/res_sip_pubsub.exports.in 389657 

Diff: https://reviewboard.asterisk.org/r/2523/diff/


Testing
-------

Subscribed to a custom presence type and upon state change made sure the appropriate provider created the expected NOTIFY body in the request.


Thanks,

Kevin Harwell

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20130523/5480f7c8/attachment.htm>


More information about the asterisk-dev mailing list