[asterisk-dev] [Code Review] SIP session timer tests for Require: timer header

[Olle E Johansson]

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


First, it would be nice to get some attribution here since I reported this bug in the existing code. Secondly, please take a look at my code for require in my PRACK branch. 

I added a more generic way to support Require headers, since if we have both session timers and PRACK and possibly something else, the Require header needs to list all of them. Your patch just solves it for timer, but will cause issues if other extensions are required. Feel free to contact me if you need help finding it.

Basically I set flags for the required extensions, like we do with supported, then add the header in a generic way for all the required extensions.

- Olle E


On Oct. 30, 2012, 6:48 p.m., Mark Michelson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/2173/
> -----------------------------------------------------------
> 
> (Updated Oct. 30, 2012, 6:48 p.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Summary
> -------
> 
> These are accompanying tests for the changes in https://reviewboard.asterisk.org/r/2172
> 
> 
> This addresses bug ASTERISK-20570.
>     https://issues.asterisk.org/jira/browse/ASTERISK-20570
> 
> 
> Diffs
> -----
> 
>   /asterisk/trunk/tests/channels/SIP/session_timers_require/configs/ast1/extensions.conf PRE-CREATION 
>   /asterisk/trunk/tests/channels/SIP/session_timers_require/configs/ast1/sip.conf PRE-CREATION 
>   /asterisk/trunk/tests/channels/SIP/session_timers_require/sipp/uac-no-refresher.xml PRE-CREATION 
>   /asterisk/trunk/tests/channels/SIP/session_timers_require/sipp/uac-no-timer-support.xml PRE-CREATION 
>   /asterisk/trunk/tests/channels/SIP/session_timers_require/sipp/uac-refresher-uac.xml PRE-CREATION 
>   /asterisk/trunk/tests/channels/SIP/session_timers_require/sipp/uac-refresher-uas.xml PRE-CREATION 
>   /asterisk/trunk/tests/channels/SIP/session_timers_require/test-config.yaml PRE-CREATION 
>   /asterisk/trunk/tests/channels/SIP/tests.yaml 3490 
> 
> Diff: https://reviewboard.asterisk.org/r/2173/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Mark
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20121031/b14b0d2f/attachment-0001.htm>