----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3455/#review11640 -----------------------------------------------------------
Ship it! Ship It! - Joshua Colp On April 16, 2014, 9:08 p.m., Jonathan Rose wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/3455/ > ----------------------------------------------------------- > > (Updated April 16, 2014, 9:08 p.m.) > > > Review request for Asterisk Developers, Joshua Colp, Matt Jordan, Mark > Michelson, opticron, and wdoekes. > > > Bugs: AST-1301 and ASTERISK-19465 > https://issues.asterisk.org/jira/browse/AST-1301 > https://issues.asterisk.org/jira/browse/ASTERISK-19465 > > > Repository: testsuite > > > Description > ------- > > Part of the effort going on with https://reviewboard.asterisk.org/r/3447/ > > The point of these tests is to establish requirements for developing a patch > to deal with the anonymity problems described on the issues. The following > charts provide the base expectations: > > for sendrpid=pai: > > | pres=allowed | pres=prohibited | > ----------------------+-------------------------+-----------------------+ > trust_id_outbound=no | PAI: 123, Privacy: none | | > ----------------------+-------------------------+-----------------------+ > trust_id_outbound=yes | PAI: 123, Privacy: none | PAI: 123, Privacy: id | > ----------------------+-------------------------+-----------------------+ > > *PAI is abbreviated a bit in this chart > > > for sendrpid=rpid: > | pres=allowed > | pres=prohibited > | > ----------------------+--------------------------------------------------------------------------------------+-----------------------------------------------------------------------------------------+ > trust_id_outbound=no | Remote-Party-ID: "123" > <sip:1...@xxx.xxx.xxx.xxx>;party=calling;privacy=off;screen=no | > Remote-Party-ID: "123" > <sip:123@anonymous.invalid>party=calling;privacy=full;screen=yes | > ----------------------+--------------------------------------------------------------------------------------+-----------------------------------------------------------------------------------------+ > trust_id_outbound=yes | Remote-Party-ID: "123" > <sip:1...@xxx.xxx.xxx.xxx>;party=calling;privacy=off;screen=no | > Remote-Party-ID: "123" > <sip:1...@xxx.xxx.xxx.xxx>;party=calling;privacy=full;screen=yes | > ----------------------+--------------------------------------------------------------------------------------+-----------------------------------------------------------------------------------------+ > > > Diffs > ----- > > /asterisk/trunk/tests/channels/SIP/tests.yaml 4958 > /asterisk/trunk/tests/channels/SIP/sendrpid/tests.yaml PRE-CREATION > /asterisk/trunk/tests/channels/SIP/sendrpid/rpid/trust/tests.yaml > PRE-CREATION > > /asterisk/trunk/tests/channels/SIP/sendrpid/rpid/trust/pres_prohib/test-config.yaml > PRE-CREATION > > /asterisk/trunk/tests/channels/SIP/sendrpid/rpid/trust/pres_prohib/sipp/send.xml > PRE-CREATION > > /asterisk/trunk/tests/channels/SIP/sendrpid/rpid/trust/pres_prohib/sipp/receive.xml > PRE-CREATION > > /asterisk/trunk/tests/channels/SIP/sendrpid/rpid/trust/pres_prohib/configs/ast1/sip.conf > PRE-CREATION > > /asterisk/trunk/tests/channels/SIP/sendrpid/rpid/trust/pres_prohib/configs/ast1/extensions.conf > PRE-CREATION > > /asterisk/trunk/tests/channels/SIP/sendrpid/rpid/trust/pres_allow/test-config.yaml > PRE-CREATION > > /asterisk/trunk/tests/channels/SIP/sendrpid/rpid/trust/pres_allow/sipp/send.xml > PRE-CREATION > > /asterisk/trunk/tests/channels/SIP/sendrpid/rpid/trust/pres_allow/sipp/receive.xml > PRE-CREATION > > /asterisk/trunk/tests/channels/SIP/sendrpid/rpid/trust/pres_allow/configs/ast1/sip.conf > PRE-CREATION > > /asterisk/trunk/tests/channels/SIP/sendrpid/rpid/trust/pres_allow/configs/ast1/extensions.conf > PRE-CREATION > /asterisk/trunk/tests/channels/SIP/sendrpid/rpid/tests.yaml PRE-CREATION > /asterisk/trunk/tests/channels/SIP/sendrpid/rpid/baseline/tests.yaml > PRE-CREATION > > /asterisk/trunk/tests/channels/SIP/sendrpid/rpid/baseline/pres_prohib/test-config.yaml > PRE-CREATION > > /asterisk/trunk/tests/channels/SIP/sendrpid/rpid/baseline/pres_prohib/sipp/send.xml > PRE-CREATION > > /asterisk/trunk/tests/channels/SIP/sendrpid/rpid/baseline/pres_prohib/sipp/receive.xml > PRE-CREATION > > /asterisk/trunk/tests/channels/SIP/sendrpid/rpid/baseline/pres_prohib/configs/ast1/sip.conf > PRE-CREATION > > /asterisk/trunk/tests/channels/SIP/sendrpid/rpid/baseline/pres_prohib/configs/ast1/extensions.conf > PRE-CREATION > > /asterisk/trunk/tests/channels/SIP/sendrpid/rpid/baseline/pres_allow/test-config.yaml > PRE-CREATION > > /asterisk/trunk/tests/channels/SIP/sendrpid/rpid/baseline/pres_allow/sipp/send.xml > PRE-CREATION > > /asterisk/trunk/tests/channels/SIP/sendrpid/rpid/baseline/pres_allow/sipp/receive.xml > PRE-CREATION > > /asterisk/trunk/tests/channels/SIP/sendrpid/rpid/baseline/pres_allow/configs/ast1/sip.conf > PRE-CREATION > > /asterisk/trunk/tests/channels/SIP/sendrpid/rpid/baseline/pres_allow/configs/ast1/extensions.conf > PRE-CREATION > /asterisk/trunk/tests/channels/SIP/sendrpid/pai/trust/tests.yaml > PRE-CREATION > > /asterisk/trunk/tests/channels/SIP/sendrpid/pai/trust/pres_prohib/test-config.yaml > PRE-CREATION > > /asterisk/trunk/tests/channels/SIP/sendrpid/pai/trust/pres_prohib/sipp/send.xml > PRE-CREATION > > /asterisk/trunk/tests/channels/SIP/sendrpid/pai/trust/pres_prohib/sipp/receive.xml > PRE-CREATION > > /asterisk/trunk/tests/channels/SIP/sendrpid/pai/trust/pres_prohib/configs/ast1/sip.conf > PRE-CREATION > > /asterisk/trunk/tests/channels/SIP/sendrpid/pai/trust/pres_prohib/configs/ast1/extensions.conf > PRE-CREATION > > /asterisk/trunk/tests/channels/SIP/sendrpid/pai/trust/pres_allow/test-config.yaml > PRE-CREATION > > /asterisk/trunk/tests/channels/SIP/sendrpid/pai/trust/pres_allow/sipp/send.xml > PRE-CREATION > > /asterisk/trunk/tests/channels/SIP/sendrpid/pai/trust/pres_allow/sipp/receive.xml > PRE-CREATION > > /asterisk/trunk/tests/channels/SIP/sendrpid/pai/trust/pres_allow/configs/ast1/sip.conf > PRE-CREATION > > /asterisk/trunk/tests/channels/SIP/sendrpid/pai/trust/pres_allow/configs/ast1/extensions.conf > PRE-CREATION > /asterisk/trunk/tests/channels/SIP/sendrpid/pai/tests.yaml PRE-CREATION > /asterisk/trunk/tests/channels/SIP/sendrpid/pai/baseline/tests.yaml > PRE-CREATION > > /asterisk/trunk/tests/channels/SIP/sendrpid/pai/baseline/pres_prohib/test-config.yaml > PRE-CREATION > > /asterisk/trunk/tests/channels/SIP/sendrpid/pai/baseline/pres_prohib/sipp/send.xml > PRE-CREATION > > /asterisk/trunk/tests/channels/SIP/sendrpid/pai/baseline/pres_prohib/sipp/receive.xml > PRE-CREATION > > /asterisk/trunk/tests/channels/SIP/sendrpid/pai/baseline/pres_prohib/configs/ast1/sip.conf > PRE-CREATION > > /asterisk/trunk/tests/channels/SIP/sendrpid/pai/baseline/pres_prohib/configs/ast1/extensions.conf > PRE-CREATION > > /asterisk/trunk/tests/channels/SIP/sendrpid/pai/baseline/pres_allow/test-config.yaml > PRE-CREATION > > /asterisk/trunk/tests/channels/SIP/sendrpid/pai/baseline/pres_allow/sipp/send.xml > PRE-CREATION > > /asterisk/trunk/tests/channels/SIP/sendrpid/pai/baseline/pres_allow/sipp/receive.xml > PRE-CREATION > > /asterisk/trunk/tests/channels/SIP/sendrpid/pai/baseline/pres_allow/configs/ast1/sip.conf > PRE-CREATION > > /asterisk/trunk/tests/channels/SIP/sendrpid/pai/baseline/pres_allow/configs/ast1/extensions.conf > PRE-CREATION > > Diff: https://reviewboard.asterisk.org/r/3455/diff/ > > > Testing > ------- > > Checked tests against the baseline... note that the prohib variant of the > rpid=pai test fails since it is including anonymized PAI data when it should > be including nothing (one of the subjects of the bug report). > > The trust variants of each test require that a patch be written to add the > trust_id_outbound setting described in comments on > https://reviewboard.asterisk.org/r/3447/ (currently in development) > > > Thanks, > > Jonathan Rose > >
-- _____________________________________________________________________ -- 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