[asterisk-dev] [Code Review] 2464: chan_gulp tests: Tests authentication failures for off nominal incoming call paths

[Mark Michelson]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/2464/#review8564
-----------------------------------------------------------

Ship it!


Aside from the problems Kinsey pointed out, this looks good.

- Mark Michelson


On April 23, 2013, 2:30 p.m., jbigelow wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/2464/
> -----------------------------------------------------------
> 
> (Updated April 23, 2013, 2:30 p.m.)
> 
> 
> Review request for Asterisk Developers and Mark Michelson.
> 
> 
> Repository: testsuite
> 
> 
> Description
> -------
> 
> This test sends various calls to res_sip for which authentication is expected to fail. This covers test #2 of the off-nominal paths listed on the test plan page of the wiki. It sends the wrong username, password, and *realm using IPv4/IPv6 and UDP/TCP. 
> 
> A small change was made to sipp.py to allow injection files to reside and be found in the sipp sub directory of the test directory.
> 
> * The realm is simply changed and the response digest is not recalculated in the Authorization header due to SIPp limitations.
> 
> 
> Diffs
> -----
> 
>   /asterisk/team/group/pimp_my_sip/lib/python/asterisk/sipp.py 3734 
>   /asterisk/team/group/pimp_my_sip/tests/channels/gulp/basic_calls/incoming/off-nominal/md5/configs/ast1/extensions.conf PRE-CREATION 
>   /asterisk/team/group/pimp_my_sip/tests/channels/gulp/basic_calls/incoming/off-nominal/md5/configs/ast1/modules.conf PRE-CREATION 
>   /asterisk/team/group/pimp_my_sip/tests/channels/gulp/basic_calls/incoming/off-nominal/md5/configs/ast1/res_sip.conf PRE-CREATION 
>   /asterisk/team/group/pimp_my_sip/tests/channels/gulp/basic_calls/incoming/off-nominal/md5/sipp/credentials-wrong_pass.csv PRE-CREATION 
>   /asterisk/team/group/pimp_my_sip/tests/channels/gulp/basic_calls/incoming/off-nominal/md5/sipp/credentials-wrong_realm.csv PRE-CREATION 
>   /asterisk/team/group/pimp_my_sip/tests/channels/gulp/basic_calls/incoming/off-nominal/md5/sipp/credentials-wrong_user.csv PRE-CREATION 
>   /asterisk/team/group/pimp_my_sip/tests/channels/gulp/basic_calls/incoming/off-nominal/md5/sipp/playback_with_initial_sdp.xml PRE-CREATION 
>   /asterisk/team/group/pimp_my_sip/tests/channels/gulp/basic_calls/incoming/off-nominal/md5/test-config.yaml PRE-CREATION 
>   /asterisk/team/group/pimp_my_sip/tests/channels/gulp/basic_calls/incoming/off-nominal/tests.yaml PRE-CREATION 
>   /asterisk/team/group/pimp_my_sip/tests/channels/gulp/basic_calls/incoming/off-nominal/userpass/configs/ast1/extensions.conf PRE-CREATION 
>   /asterisk/team/group/pimp_my_sip/tests/channels/gulp/basic_calls/incoming/off-nominal/userpass/configs/ast1/modules.conf PRE-CREATION 
>   /asterisk/team/group/pimp_my_sip/tests/channels/gulp/basic_calls/incoming/off-nominal/userpass/configs/ast1/res_sip.conf PRE-CREATION 
>   /asterisk/team/group/pimp_my_sip/tests/channels/gulp/basic_calls/incoming/off-nominal/userpass/sipp/credentials-wrong_pass.csv PRE-CREATION 
>   /asterisk/team/group/pimp_my_sip/tests/channels/gulp/basic_calls/incoming/off-nominal/userpass/sipp/credentials-wrong_realm.csv PRE-CREATION 
>   /asterisk/team/group/pimp_my_sip/tests/channels/gulp/basic_calls/incoming/off-nominal/userpass/sipp/credentials-wrong_user.csv PRE-CREATION 
>   /asterisk/team/group/pimp_my_sip/tests/channels/gulp/basic_calls/incoming/off-nominal/userpass/sipp/playback_with_initial_sdp.xml PRE-CREATION 
>   /asterisk/team/group/pimp_my_sip/tests/channels/gulp/basic_calls/incoming/off-nominal/userpass/test-config.yaml PRE-CREATION 
>   /asterisk/team/group/pimp_my_sip/tests/channels/gulp/basic_calls/incoming/tests.yaml 3734 
> 
> Diff: https://reviewboard.asterisk.org/r/2464/diff/
> 
> 
> Testing
> -------
> 
> * Ensured that all 24 combinations fail as expected.
> 
> 
> Thanks,
> 
> jbigelow
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20130509/b077bde9/attachment-0001.htm>