----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3454/#review11630 -----------------------------------------------------------
http://svn.asterisk.org/svn/asterisk/trunk/res/res_rtp_asterisk.c <https://reviewboard.asterisk.org/r/3454/#comment21400> This is going to make Asterisk always assume that it is the controlling session, which is incorrect. If the UNKNOWN role is not sufficient, then the signature of ice_create would have to change to know whether or not Asterisk is creating the ICE session for an offer or for an answer. That information has to be passed down to res_rtp_asterisk from whatever is creating the RTP instance. - Matt Jordan On April 16, 2014, 3:43 a.m., Marko Seidenglanz wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/3454/ > ----------------------------------------------------------- > > (Updated April 16, 2014, 3:43 a.m.) > > > Review request for Asterisk Developers. > > > Bugs: ASTERISK-23026 > https://issues.asterisk.org/jira/browse/ASTERISK-23026 > > > Repository: Asterisk > > > Description > ------- > > The following patch must be applied to res_rtp_asterisk to make outgoing > calls to WebRTC capable browsers working. This is necessary due to the > following issue in role negotiation: > https://code.google.com/p/webrtc/issues/detail?id=2717&can=4&colspec=ID%20Pri%20Mstone%20ReleaseBlock%20Area%20Status%20Owner%20Summary > > > Diffs > ----- > > http://svn.asterisk.org/svn/asterisk/trunk/res/res_rtp_asterisk.c 412438 > > Diff: https://reviewboard.asterisk.org/r/3454/diff/ > > > Testing > ------- > > > Thanks, > > Marko Seidenglanz > >
-- _____________________________________________________________________ -- Bandwidth and Colocation Provided by http://www.api-digital.com -- asterisk-dev mailing list To UNSUBSCRIBE or update options visit: http://lists.digium.com/mailman/listinfo/asterisk-dev
