[asterisk-dev] [Code Review] ICE, STUN, and TURN Support

[Joshua Colp]

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

(Updated May 9, 2012, 8:46 a.m.)


Review request for Asterisk Developers.


Changes
-------

Updated with incorporated review feedback changes.


Summary
-------

This patch adds support for the following:

ICE attribute parsing and generation in chan_sip
Usage of the ICE interface in chan_sip
ICE support within res_rtp_asterisk
STUN support within res_rtp_asterisk for getting the server reflexive address
TURN support within res_rtp_asterisk for relaying traffic when needed
Additional configuration options for the above

One area which could use some feedback is the pjproject integration. Should we try to use their build system as I have done to integrate things, or try to roll their stuff into ours further?

If you would like some background on ICE and the actual support here take a gander at the wiki page located at https://wiki.asterisk.org/wiki/display/~jcolp/ICE%2C+STUN%2C+and+TURN+Support - hopefully it proves useful to you.


Diffs (updated)
-----

  /trunk/channels/chan_sip.c 361952 
  /trunk/configs/rtp.conf.sample 361952 
  /trunk/include/asterisk/rtp_engine.h 361952 
  /trunk/main/rtp_engine.c 361952 
  /trunk/res/Makefile 361952 
  /trunk/res/res_rtp_asterisk.c 361952 

Diff: https://reviewboard.asterisk.org/r/1891/diff


Testing
-------

Tested various scenarios using Bria and X-Lite behind multiple NATs, different NATs, different internet connections but locally reachable, etc. It works well.


Thanks,

Joshua

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20120509/82c97173/attachment.htm>