[asterisk-dev] [Code Review] 2829: Testsuite: Tests for outbound PJSIP authentication

[George Joseph]

Can I suggest 1 more test?   Since I had to change where the default of
"asterisk" is set for outbound 401/407 challenges, a test to make sure the
default is still set correctly would probably be good.  I'll sleep better.
:)


On Fri, Sep 6, 2013 at 12:42 PM, Mark Michelson <reviewboard at asterisk.org>wrote:

>    This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/2829/
>   Review request for Asterisk Developers.
> By Mark Michelson.
>
> *Updated Sept. 6, 2013, 6:42 p.m.*
> Changes
>
> Updated description in test-config.yaml to be more accurate.
>
>   *Bugs: * ASTERISK-22386<https://issues.asterisk.org/jira/browse/ASTERISK-22386>,
> ASTERISK-22471 <https://issues.asterisk.org/jira/browse/ASTERISK-22471>
>  *Repository: * testsuite
> Description
>
> On ASTERISK-22471, George Joseph created a patch that eases the burden of outbound authentication. If no realm is specified in an auth section, then we will simply mirror the realm presented in the 401/407 response that we receive.
>
> Since this patch is a candidate for Asterisk 12, it is required to have test coverage. This set of tests exercises outbound authentication. The test starts up two Asterisk instances, uut and bob. uut places four outbound calls to bob. Bob challenges each of the four calls the exact same way; however, the authentication is configured four different ways on uut. uut is expected to be able to respond to each of the four challenges correctly.
>
> The four methods of authentication configuration are:
> 1. Userpass authentication, explicit realm present.
> 2. Userpass authentication, no realm specified.
> 3. MD5 authentication, explicit realm present.
> 4. MD5 authentictaion, no realm specified.
>
>   Testing
>
> The tests pass with the patch from ASTERISK-22471 applied. Without the patch, cases 2 and 4 are unable to authenticate properly.
>
>   Diffs (updated)
>
>    - /asterisk/trunk/tests/channels/pjsip/basic_calls/outgoing/nominal/auth/configs/ast1/extensions.conf
>    (PRE-CREATION)
>    - /asterisk/trunk/tests/channels/pjsip/basic_calls/outgoing/nominal/auth/configs/ast1/pjsip.conf
>    (PRE-CREATION)
>    - /asterisk/trunk/tests/channels/pjsip/basic_calls/outgoing/nominal/auth/configs/ast2/extensions.conf
>    (PRE-CREATION)
>    - /asterisk/trunk/tests/channels/pjsip/basic_calls/outgoing/nominal/auth/configs/ast2/pjsip.conf
>    (PRE-CREATION)
>    - /asterisk/trunk/tests/channels/pjsip/basic_calls/outgoing/nominal/auth/test-config.yaml
>    (PRE-CREATION)
>    - /asterisk/trunk/tests/channels/pjsip/basic_calls/outgoing/nominal/tests.yaml
>    (4126)
>
> View Diff <https://reviewboard.asterisk.org/r/2829/diff/>
>
> --
> _____________________________________________________________________
> -- 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
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20130906/203667bd/attachment.htm>