[asterisk-dev] [Code Review] Help mitigate reinvite glares: Attempt 2

[Kevin Fleming]

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

Ship it!


Other than my minor nits, this looks great.

- Kevin


On May 30, 2012, 3:35 p.m., Mark Michelson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/1954/
> -----------------------------------------------------------
> 
> (Updated May 30, 2012, 3:35 p.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Summary
> -------
> 
> This is similar but not quite the same as my previous review request.  The goal of this setting is to prevent reinvite glare scenarios that arise when multiple Asterisk servers are in a path where direct media will be used. The most common glare scenario arises because the two Asterisk servers immediately send direct media reinvites to each other once the initial INVITE transaction has completed. 
> 
> This review request adds a new option for the directmedia setting, called "outgoing". If set, then Asterisk will only transmit an immediate direct media reinvite if the call direction is outgoing. The key word here is "immediate." Asterisk will still be able to send direct media reinvites on incoming calls, but they will only be sent in reaction to changes in the remote RTP address.
> 
> 
> Diffs
> -----
> 
>   /trunk/channels/chan_sip.c 367639 
>   /trunk/channels/sip/include/sip.h 367639 
>   /trunk/configs/sip.conf.sample 367639 
> 
> Diff: https://reviewboard.asterisk.org/r/1954/diff
> 
> 
> Testing
> -------
> 
> This has been tested in a scenario with two phones and two Asterisk servers. I have verified that with this set, direct media is properly set up between the two phones and I saw no glares in many calls (around 20)
> 
> 
> Thanks,
> 
> Mark
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20120530/253a4f98/attachment.htm>