----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/4185/#review13833 -----------------------------------------------------------
branches/12/main/sorcery.c <https://reviewboard.asterisk.org/r/4185/#comment24302> This will only ever get called when a name match was not found, so you don't need to also look here. Just skip if no regex. - Joshua Colp On Nov. 20, 2014, 6:17 p.m., George Joseph wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/4185/ > ----------------------------------------------------------- > > (Updated Nov. 20, 2014, 6:17 p.m.) > > > Review request for Asterisk Developers, Joshua Colp and Mark Michelson. > > > Bugs: ASTERISK-24520 > https://issues.asterisk.org/jira/browse/ASTERISK-24520 > > > Repository: Asterisk > > > Description > ------- > > As a result of https://reviewboard.asterisk.org/r/3305, res_sorcery_realtime > was tossing database fields that didn't have an exact match to a sorcery > registered field. This broke the ability to use regexes as field names which > manifested itself as a failure of res_pjsip_phoneprov_provider which uses > this capability. It also broke handling of fields that start with '@' in > realtime but I don't think anyone noticed. > > This patch does the following... > Modifies ast_sorcery_fields_register to pre-compile the name regex. > Modifies ast_sorcery_is_object_field_registered to test the regex if it > exists instead of doing an exact strcmp. > Modifies res_pjsip_phoneprov_provider with a few tweaks to get it to work > with realtime. > > > Diffs > ----- > > branches/12/tests/test_sorcery.c 428504 > branches/12/res/res_pjsip_phoneprov_provider.c 428504 > branches/12/main/sorcery.c 428504 > > Diff: https://reviewboard.asterisk.org/r/4185/diff/ > > > Testing > ------- > > Tested by me with realtime phoneprov, aor and registration objects both with > and without registered objects. > Testsuite testing in progress. EDIT: complete. No issues. > Will also be tested by John Kiniston who reported the > res_pjsip_phoneprov_provider issue. EDIT: complete. > > > Thanks, > > George Joseph > >
-- _____________________________________________________________________ -- 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