[asterisk-dev] [Code Review] 4499: Support in dialog OPTIONS

Joshua Colp reviewboard at asterisk.org
Tue Mar 17 09:53:38 CDT 2015


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



/trunk/res/res_pjsip_dlg_options.c
<https://reviewboard.asterisk.org/r/4499/#comment25282>

    This should mirror send_options_response in res/res_pjsip/pjsip_options.c and include additional headers:
    
    " An OPTIONS request received within a dialog generates a 200 (OK)
       response that is identical to one constructed outside a dialog and
       does not have any impact on the dialog."


- Joshua Colp


On March 15, 2015, 8:02 a.m., yaron nahum wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/4499/
> -----------------------------------------------------------
> 
> (Updated March 15, 2015, 8:02 a.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Bugs: ASTERISK-24862
>     https://issues.asterisk.org/jira/browse/ASTERISK-24862
> 
> 
> Repository: Asterisk
> 
> 
> Description
> -------
> 
> Respond to OPTIONS message sent on an existing dialog with 200OK. 
> This feature is vital in order to interoperate with some switches that send OPTIONS message periodically per active call to make sure it is still alive. Not responding would cause the switch to disconnect the call. 
> This functionality used to work on the old SIP channel, but was not implemented on PJSIP.
> 
> 
> Diffs
> -----
> 
>   /trunk/res/res_pjsip_dlg_options.c PRE-CREATION 
> 
> Diff: https://reviewboard.asterisk.org/r/4499/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> yaron nahum
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20150317/6e450718/attachment.html>


More information about the asterisk-dev mailing list