[asterisk-dev] [Code Review] 2547: Remove single event type restriction from res_sip_pubsub

Kevin Harwell reviewboard at asterisk.org
Wed May 22 16:19:08 CDT 2013


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



/team/group/pimp_my_sip/res/res_sip_pubsub.c
<https://reviewboard.asterisk.org/r/2547/#comment17053>

    This won't work as pjsip_evsub_register_pkg calls pjsip_endpt_add_capability which will assert on count of 0 and/or null accept.


- Kevin Harwell


On May 15, 2013, 10:15 p.m., Joshua Colp wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/2547/
> -----------------------------------------------------------
> 
> (Updated May 15, 2013, 10:15 p.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Repository: Asterisk
> 
> 
> Description
> -------
> 
> The pjsip subscription system has a restriction where only a single event type can be registered. Due to the way res_sip_pubsub has been architected this really doesn't matter as it is always called into and ultimately finds a respective handler among all registered. The change attached moves a bit of logic into res_sip_pubsub (adding of accept types to the global Accept header) and only registers an event type once. By doing so res_sip_pubsub now supports multiple modules registering the same event type.
> 
> 
> Diffs
> -----
> 
>   /team/group/pimp_my_sip/res/res_sip_pubsub.c 388882 
> 
> Diff: https://reviewboard.asterisk.org/r/2547/diff/
> 
> 
> Testing
> -------
> 
> Set MWI up, confirmed it works as expected still.
> 
> 
> Thanks,
> 
> Joshua Colp
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20130522/1eb63898/attachment.htm>


More information about the asterisk-dev mailing list