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

[opticron]

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



/asterisk/team/group/pimp_my_sip/tests/channels/gulp/basic_calls/incoming/off-nominal/md5/configs/ast1/extensions.conf
<https://reviewboard.asterisk.org/r/2464/#comment16247>

    I'd recommend using Echo() here so the test isn't time-constrained in dialplan.



/asterisk/team/group/pimp_my_sip/tests/channels/gulp/basic_calls/incoming/off-nominal/md5/configs/ast1/modules.conf
<https://reviewboard.asterisk.org/r/2464/#comment16248>

    If you choose to use Echo, you can drop app_playback and format_gsm from this list (probably codec_gsm as well).



/asterisk/team/group/pimp_my_sip/tests/channels/gulp/basic_calls/incoming/off-nominal/md5/configs/ast1/res_sip.conf
<https://reviewboard.asterisk.org/r/2464/#comment16249>

    You have a duplicate transport context name here even though the templates used vary.



/asterisk/team/group/pimp_my_sip/tests/channels/gulp/basic_calls/incoming/off-nominal/md5/sipp/playback_with_initial_sdp.xml
<https://reviewboard.asterisk.org/r/2464/#comment16250>

    This is invalid XML (flagged by reviewboard as well).



/asterisk/team/group/pimp_my_sip/tests/channels/gulp/basic_calls/incoming/off-nominal/md5/sipp/playback_with_initial_sdp.xml
<https://reviewboard.asterisk.org/r/2464/#comment16251>

    Idem.


The comments above apply to the respective files in both tests.

- opticron


On April 23, 2013, 9:30 a.m., jbigelow wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/2464/
> -----------------------------------------------------------
> 
> (Updated April 23, 2013, 9:30 a.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/20130502/3917f65f/attachment-0001.htm>