----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3577/#review12011 -----------------------------------------------------------
Ship it! Ship It! - Matt Jordan On May 30, 2014, 8:58 a.m., Joshua Colp wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/3577/ > ----------------------------------------------------------- > > (Updated May 30, 2014, 8:58 a.m.) > > > Review request for Asterisk Developers. > > > Bugs: ASTERISK-23541 > https://issues.asterisk.org/jira/browse/ASTERISK-23541 > > > Repository: Asterisk > > > Description > ------- > > The bridge_native_rtp module currently uses the bridge result of the first > channel that joins a bridge as the ultimate result. This means that if the > first channel has direct media enabled but the second does not a direct media > bridge will still occur. > > This change makes it so that both sides are taken into account. If either > side forbids the bridge or responds with a local bridge result then either a > generic or local bridge occurs. > > > Diffs > ----- > > /branches/12/bridges/bridge_native_rtp.c 414813 > > Diff: https://reviewboard.asterisk.org/r/3577/diff/ > > > Testing > ------- > > Configured two SIP endpoints: caller with direct media enabled and callee > with direct media disabled. Placed a call from the caller to the callee. > > Without this patch a reinvite occurs and the media is direct. > With this patch no reinvite occurs and media is locally exchanged. > > > Thanks, > > Joshua Colp > >
-- _____________________________________________________________________ -- 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
