----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3093/#review10668 -----------------------------------------------------------
Ship it! /branches/12/res/res_pjsip_dialog_info.c <https://reviewboard.asterisk.org/r/3093/#comment20158> These could be replaced with ast_str_truncate(). - opticron On Jan. 15, 2014, 8:35 a.m., Joshua Colp wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/3093/ > ----------------------------------------------------------- > > (Updated Jan. 15, 2014, 8:35 a.m.) > > > Review request for Asterisk Developers. > > > Bugs: ASTERISK-21443 > https://issues.asterisk.org/jira/browse/ASTERISK-21443 > > > Repository: Asterisk > > > Description > ------- > > Some devices use the dialog-info+xml to get presence information about an > extension. The attached change implements this but DOES NOT implement the > "pickup ringing channel" functionality available on Snoms. > > To facilitate this implementation I have also changed it so the subscription > is passed to the body creation callback. This allows implementations to store > state information on the subscription. > > > Diffs > ----- > > /branches/12/res/res_pjsip_pidf.c 405576 > /branches/12/res/res_pjsip_exten_state.c 405576 > /branches/12/res/res_pjsip_dialog_info.c PRE-CREATION > /branches/12/include/asterisk/res_pjsip_exten_state.h 405576 > /branches/12/UPGRADE.txt 405576 > > Diff: https://reviewboard.asterisk.org/r/3093/diff/ > > > Testing > ------- > > Subscribed to an extension from a device that uses dialog-info+xml. Confirmed > NOTIFY messages going out contain the expected content. > > > 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
