> On Oct. 13, 2014, 11:18 a.m., Matt Jordan wrote: > > /trunk/res/res_pjsip_sdp_rtp.c, lines 1097-1103 > > <https://reviewboard.asterisk.org/r/3992/diff/2/?file=67982#file67982line1097> > > > > Blobs! > > Joshua Colp wrote: > Per other review this needs alembic.
And you're correct on the other review, this actually should be CHANGES. - Matt ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3992/#review13497 ----------------------------------------------------------- On Oct. 10, 2014, 3:13 p.m., Joshua Colp wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/3992/ > ----------------------------------------------------------- > > (Updated Oct. 10, 2014, 3:13 p.m.) > > > Review request for Asterisk Developers. > > > Repository: Asterisk > > > Description > ------- > > When enabling SRTP support in PJSIP it is either forced on or disabled. This > means that if you specify SRTP but the client does not support it the session > will fail. For situations where this guarantee is not required this new > functionality can be used to optimistically use SRTP if possible. This has > the added benefit of encrypting the media when possible but does not > guarantee it. This also fixes an issue where a client may offer SRTP using > the normal transport but we reject it. > > > Diffs > ----- > > /trunk/res/res_pjsip_session.c 425156 > /trunk/res/res_pjsip_sdp_rtp.c 425156 > /trunk/res/res_pjsip/pjsip_configuration.c 425156 > /trunk/res/res_pjsip.c 425156 > /trunk/include/asterisk/res_pjsip_session.h 425156 > /trunk/include/asterisk/res_pjsip.h 425156 > /trunk/configs/samples/pjsip.conf.sample 425156 > > Diff: https://reviewboard.asterisk.org/r/3992/diff/ > > > Testing > ------- > > Used Blink to place calls with optimistic enabled and disabled on the PJSIP > side. > In Blink I alternated between disabled/mandatory/optional. > Confirmed that for each scenario the expected outcome occurred. > > Blink Asterisk Result > Disabled Optimistic Off Failed > Disabled Optimistic On Success (Not encrypted) > Mandatory Optimistic Off Success (Encrypted) > Mandatory Optimistic On Success (Encrypted) > Optional Optimistic Off Success (Encrypted) > Optional Optimistic On Success (Encrypted) > > > 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