----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/4346/#review14209 -----------------------------------------------------------
/branches/11/channels/chan_sip.c <https://reviewboard.asterisk.org/r/4346/#comment24663> \retval /branches/11/channels/chan_sip.c <https://reviewboard.asterisk.org/r/4346/#comment24664> I'd rather see this code moved/duplicated into uas_sips_contact and uac_sips_contact. Right now it looked like you missed half the criteria when looking at those functions. /branches/11/channels/chan_sip.c <https://reviewboard.asterisk.org/r/4346/#comment24665> Can this be safely removed? Does the contact get added somewhere else? Is there not a reason why it's done right after the ast_sip_ouraddrfor? /branches/11/channels/chan_sip.c <https://reviewboard.asterisk.org/r/4346/#comment24666> Indentation of the comment. - wdoekes On Jan. 15, 2015, 8:07 p.m., Mark Michelson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/4346/ > ----------------------------------------------------------- > > (Updated Jan. 15, 2015, 8:07 p.m.) > > > Review request for Asterisk Developers. > > > Bugs: ASTERISK-24646 > https://issues.asterisk.org/jira/browse/ASTERISK-24646 > > > Repository: Asterisk > > > Description > ------- > > Please see the description on /r/4345 for a description of the > interoperability problem. > > This patch is the chan_sip version of the patch. The chan_sip patch is a bit > more weird since creation of Contact headers is not divided into dialog vs. > non-dialog situations. For this patch, I have expanded the build_contact() > function to base the scheme of the Contact header based on the rules of RFC > 3261 sections 8.1.1.8 and 12.1.1. To do this, I have added two parameters so > that it is clear whether the Contact header is being created due to a > response to an incoming request or if the Contact is being added to a created > outgoing request. > > > Diffs > ----- > > /branches/11/channels/chan_sip.c 428405 > > Diff: https://reviewboard.asterisk.org/r/4346/diff/ > > > Testing > ------- > > This patch has been tested by the reporter of ASTERISK-24646 and has been > confirmed to solve the interop issue. > > > Thanks, > > Mark Michelson > >
-- _____________________________________________________________________ -- 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
