<p>Richard Mudgett has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.asterisk.org/6842">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">rtp/strict_rtp: Be forgiving for packet count trigger.<br><br>On my test box, the test frequently fails because the RTCPReceived<br>SentPacket count is not what is expected to trigger the RTP attack stream.<br><br>* Use a range of allowed packet counts to trigger the RTP attack stream.<br><br>* Added a message to tell how many SentPackets triggered the attack stream<br>for each AMI connection.<br><br>Change-Id: I58b2d5938cdcd165a4954429c70de34fb1184cd5<br>---<br>M tests/rtp/strict_rtp/strict_rtp.py<br>1 file changed, 5 insertions(+), 1 deletion(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.asterisk.org:29418/testsuite refs/changes/42/6842/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/tests/rtp/strict_rtp/strict_rtp.py b/tests/rtp/strict_rtp/strict_rtp.py<br>index a782152..57974b5 100644<br>--- a/tests/rtp/strict_rtp/strict_rtp.py<br>+++ b/tests/rtp/strict_rtp/strict_rtp.py<br>@@ -75,8 +75,12 @@<br>         ami   The AMI protocol instance<br>         event The Newchannel event<br>         """<br>-        if event['sentpackets'] != '250':<br>+        if int(event['sentpackets']) < 240 or 260 < int(event['sentpackets']):<br>             return<br>+        LOGGER.info("AMI " +<br>+                    str(ami.id) +<br>+                    ": Triggered on AMI RTCPReceived event with SentPackets: " +<br>+                    event['sentpackets'])<br> <br>         self.test_object.set_passed(True)<br>         protocol = StrictRtpTester.NoAnswerProtocol(self.test_object)<br></pre><p>To view, visit <a href="https://gerrit.asterisk.org/6842">change 6842</a>. To unsubscribe, visit <a href="https://gerrit.asterisk.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.asterisk.org/6842"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: testsuite </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>
<div style="display:none"> Gerrit-Change-Id: I58b2d5938cdcd165a4954429c70de34fb1184cd5 </div>
<div style="display:none"> Gerrit-Change-Number: 6842 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Richard Mudgett <rmudgett@digium.com> </div>