[asterisk-dev] [Code Review] sip_tls_call test added to external test suite

[Paul Belanger]

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


A few comments on binding to the loopback adapter rather then 0.0.0.0.


/asterisk/trunk/tests/channels/SIP/sip_tls_call/configs/ast1/sip.conf
<https://reviewboard.asterisk.org/r/1276/#comment7549>

    updbindaddr=127.0.0.1:5060



/asterisk/trunk/tests/channels/SIP/sip_tls_call/configs/ast1/sip.conf
<https://reviewboard.asterisk.org/r/1276/#comment7548>

    This can be removed



/asterisk/trunk/tests/channels/SIP/sip_tls_call/configs/ast1/sip.conf
<https://reviewboard.asterisk.org/r/1276/#comment7550>

    tlsbindaddr=127.0.0.1:5060



/asterisk/trunk/tests/channels/SIP/sip_tls_call/configs/ast1/sip.conf
<https://reviewboard.asterisk.org/r/1276/#comment7551>

    Would be nice to secret=blabblah, to test authentication too.



/asterisk/trunk/tests/channels/SIP/sip_tls_call/configs/ast2/sip.conf
<https://reviewboard.asterisk.org/r/1276/#comment7552>

    udpbindaddr=127.0.0.2:5060



/asterisk/trunk/tests/channels/SIP/sip_tls_call/configs/ast2/sip.conf
<https://reviewboard.asterisk.org/r/1276/#comment7553>

    Can be removed



/asterisk/trunk/tests/channels/SIP/sip_tls_call/configs/ast2/sip.conf
<https://reviewboard.asterisk.org/r/1276/#comment7554>

    tlsbindaddr=127.0.0.2:5060



/asterisk/trunk/tests/channels/SIP/sip_tls_call/configs/ast2/sip.conf
<https://reviewboard.asterisk.org/r/1276/#comment7555>

    Again, same comment about secret


- Paul


On 2011-06-21 12:33:58, jrose wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/1276/
> -----------------------------------------------------------
> 
> (Updated 2011-06-21 12:33:58)
> 
> 
> Review request for Asterisk Developers, Russell Bryant, David Vossel, and Paul Belanger.
> 
> 
> Summary
> -------
> 
> First, you can ignore the text files.  spacespacespace.txt was just used while I was working out shell command stuff and I accidentally added it to this diff and I've also removed test-output.txt.
> 
> I'm still not perfectly sure how this is going to work with the cert files.  I'd guess it'll be fine regardless of the machine, but I haven't tested this on another machine yet that didn't create these cert files.
> 
> This test uses the basic-call test in IAX2 as a base.
> 
> 
> Diffs
> -----
> 
>   /asterisk/trunk/tests/channels/SIP/sip_tls_call/configs/ast1/extensions.conf PRE-CREATION 
>   /asterisk/trunk/tests/channels/SIP/sip_tls_call/configs/ast1/manager.general.conf.inc PRE-CREATION 
>   /asterisk/trunk/tests/channels/SIP/sip_tls_call/configs/ast1/sip.conf PRE-CREATION 
>   /asterisk/trunk/tests/channels/SIP/sip_tls_call/configs/ast2/extensions.conf PRE-CREATION 
>   /asterisk/trunk/tests/channels/SIP/sip_tls_call/configs/ast2/manager.general.conf.inc PRE-CREATION 
>   /asterisk/trunk/tests/channels/SIP/sip_tls_call/configs/ast2/sip.conf PRE-CREATION 
>   /asterisk/trunk/tests/channels/SIP/sip_tls_call/configs/helper1 PRE-CREATION 
>   /asterisk/trunk/tests/channels/SIP/sip_tls_call/configs/helper2 PRE-CREATION 
>   /asterisk/trunk/tests/channels/SIP/sip_tls_call/configs/keys/ca.cfg PRE-CREATION 
>   /asterisk/trunk/tests/channels/SIP/sip_tls_call/configs/keys/ca.crt PRE-CREATION 
>   /asterisk/trunk/tests/channels/SIP/sip_tls_call/configs/keys/ca.key PRE-CREATION 
>   /asterisk/trunk/tests/channels/SIP/sip_tls_call/configs/keys/serverA.crt PRE-CREATION 
>   /asterisk/trunk/tests/channels/SIP/sip_tls_call/configs/keys/serverA.csr PRE-CREATION 
>   /asterisk/trunk/tests/channels/SIP/sip_tls_call/configs/keys/serverA.key PRE-CREATION 
>   /asterisk/trunk/tests/channels/SIP/sip_tls_call/configs/keys/serverA.pem PRE-CREATION 
>   /asterisk/trunk/tests/channels/SIP/sip_tls_call/configs/keys/serverB.crt PRE-CREATION 
>   /asterisk/trunk/tests/channels/SIP/sip_tls_call/configs/keys/serverB.csr PRE-CREATION 
>   /asterisk/trunk/tests/channels/SIP/sip_tls_call/configs/keys/serverB.key PRE-CREATION 
>   /asterisk/trunk/tests/channels/SIP/sip_tls_call/configs/keys/serverB.pem PRE-CREATION 
>   /asterisk/trunk/tests/channels/SIP/sip_tls_call/configs/keys/tmp.cfg PRE-CREATION 
>   /asterisk/trunk/tests/channels/SIP/sip_tls_call/run-test PRE-CREATION 
>   /asterisk/trunk/tests/channels/SIP/sip_tls_call/test-config.yaml PRE-CREATION 
>   /asterisk/trunk/tests/channels/SIP/tests.yaml 1657 
>   /asterisk/trunk/tests/tests.yaml 1657 
> 
> Diff: https://reviewboard.asterisk.org/r/1276/diff
> 
> 
> Testing
> -------
> 
> How did I test the test?  Mostly by checking the sip debug logs to make sure the call was going through as a SIP/TLS call.  The usual flow of SIP messages was there and it resembled my regular SIP/TLS calls.
> 
> 
> Thanks,
> 
> jrose
> 
>

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