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

Joshua Colp reviewboard at asterisk.org
Tue May 29 14:16:19 CDT 2012

This is an automatically generated e-mail. To reply, visit:

(Updated May 29, 2012, 2:16 p.m.)

Review request for Asterisk Developers.


Incorporated latest review feedback.


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


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



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

More information about the asterisk-dev mailing list