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

[Mark Michelson]

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

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/c8700fd9/attachment-0001.htm>