[asterisk-dev] [Code Review] 4274: Testsuite: Ensure that outbound registrations with bad auth do not result in an infinite loop.

Kevin Harwell reviewboard at asterisk.org
Tue Dec 16 17:36:41 CST 2014


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



/asterisk/trunk/tests/channels/pjsip/registration/outbound/auth_loop/sipp/uas-reg-401.xml
<https://reviewboard.asterisk.org/r/4274/#comment24491>

    It doesn't look like you are using "rrs", so this comment can be removed.


Would it make any sense to check for the AMI Registry event in this scenario?  For instance two registration attempts should raise two Registry events. More than that of course would result in an error.

- Kevin Harwell


On Dec. 16, 2014, 4:22 p.m., Mark Michelson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/4274/
> -----------------------------------------------------------
> 
> (Updated Dec. 16, 2014, 4:22 p.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Repository: testsuite
> 
> 
> Description
> -------
> 
> This is a test for the contents of review 4273.
> 
> This test sets up a SIPp scenario that expects two REGISTER requests to be sent from Asterisk. Each of these REGISTER requests is responded to with a 401. Asterisk should completely cease sending registrations after the second 401 is received. The SIPp scenario stays alive a little after the final 401 is sent to ensure that Asterisk does not send any further REGISTER requests.
> 
> 
> Diffs
> -----
> 
>   /asterisk/trunk/tests/channels/pjsip/registration/outbound/tests.yaml 6075 
>   /asterisk/trunk/tests/channels/pjsip/registration/outbound/auth_loop/test-config.yaml PRE-CREATION 
>   /asterisk/trunk/tests/channels/pjsip/registration/outbound/auth_loop/sipp/uas-reg-401.xml PRE-CREATION 
>   /asterisk/trunk/tests/channels/pjsip/registration/outbound/auth_loop/configs/ast1/pjsip.conf PRE-CREATION 
> 
> Diff: https://reviewboard.asterisk.org/r/4274/diff/
> 
> 
> Testing
> -------
> 
> This test fails without the changes from /r/4273. The test succeeds with the changes applied.
> 
> 
> Thanks,
> 
> Mark Michelson
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20141216/07a98ffe/attachment.html>


More information about the asterisk-dev mailing list