----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/4455/#review14578 -----------------------------------------------------------
branches/13/res/res_pjsip/config_global.c <https://reviewboard.asterisk.org/r/4455/#comment25127> I think the default order should be ip,username,anonymous. Matching on IP address should supersede matching on a username since an IP match is more likely to be desired than a username match if an IP match has been set up for an endpoint. - Mark Michelson On March 2, 2015, 8:04 p.m., Kevin Harwell wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/4455/ > ----------------------------------------------------------- > > (Updated March 2, 2015, 8:04 p.m.) > > > Review request for Asterisk Developers. > > > Bugs: ASTERISK-24840 > https://issues.asterisk.org/jira/browse/ASTERISK-24840 > > > Repository: Asterisk > > > Description > ------- > > It's possible to have a scenario that will create a conflict between endpoint > identifiers. For instance an incoming call could be identified by two > different endpoint identifiers and the one chosen depended upon which > identifier module loaded first. This of course causes problems when, for > example, the incoming call is expected to be identified by username, but > instead is identified by ip. This patch adds a new 'global' option to > res_pjsip called 'identify_by_priority'. It is a comma separated list of > endpoint identifier names that specifies the order by which identifiers are > processed and checked. > > > Diffs > ----- > > branches/13/res/res_pjsip_endpoint_identifier_user.c 432422 > branches/13/res/res_pjsip_endpoint_identifier_ip.c 432422 > branches/13/res/res_pjsip_endpoint_identifier_anonymous.c 432422 > branches/13/res/res_pjsip/config_global.c 432422 > branches/13/res/res_pjsip.c 432422 > branches/13/include/asterisk/res_pjsip.h 432422 > > branches/13/contrib/ast-db-manage/config/versions/45e3f47c6c44_add_pjsip_identify_by_priority.py > PRE-CREATION > branches/13/configs/samples/pjsip.conf.sample 432422 > branches/13/CHANGES 432422 > > Diff: https://reviewboard.asterisk.org/r/4455/diff/ > > > Testing > ------- > > Added a testsuite test: https://reviewboard.asterisk.org/r/4456/ > > > Thanks, > > Kevin Harwell > >
-- _____________________________________________________________________ -- 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