<p>Joshua Colp <strong>merged</strong> this change.</p><p><a href="https://gerrit.asterisk.org/7437">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  Joshua Colp: Looks good to me, but someone else must approve; Approved for Submit
  Sean Bright: Looks good to me, but someone else must approve
  Kevin Harwell: Looks good to me, approved

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">res_rtp_asterisk.c: Increase strictrtp learning timeout time.<br><br>More complicated direct media reinvite negotiations can result in longer<br>delays before direct media flows.  The strictrtp learning timeout time<br>was too short.  One log showed that the first RTP packet came in just<br>after three seconds.<br><br>* Increase the strictrtp learning timeout time from 1.5 to 5 seconds.<br><br>ASTERISK-27453<br><br>Change-Id: Ic5e711164cbb91b4d1c1e40c83697755640f138c<br>---<br>M res/res_rtp_asterisk.c<br>1 file changed, 8 insertions(+), 1 deletion(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/res/res_rtp_asterisk.c b/res/res_rtp_asterisk.c<br>index 2730611..5aeb791 100644<br>--- a/res/res_rtp_asterisk.c<br>+++ b/res/res_rtp_asterisk.c<br>@@ -140,7 +140,14 @@<br>         STRICT_RTP_CLOSED,   /*! Drop all RTP packets not coming from source that was learned */<br> };<br> <br>-#define STRICT_RTP_LEARN_TIMEOUT     1500    /*!< milliseconds */<br>+/*!<br>+ * \brief Strict RTP learning timeout time in milliseconds<br>+ *<br>+ * \note Set to 5 seconds to allow reinvite chains for direct media<br>+ * to settle before media actually starts to arrive.  There may be a<br>+ * reinvite collision involved on the other leg.<br>+ */<br>+#define STRICT_RTP_LEARN_TIMEOUT        5000<br> <br> #define DEFAULT_STRICT_RTP -1 /*!< Enabled */<br> #define DEFAULT_ICESUPPORT 1<br></pre><p>To view, visit <a href="https://gerrit.asterisk.org/7437">change 7437</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/7437"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: merged </div>
<div style="display:none"> Gerrit-Change-Id: Ic5e711164cbb91b4d1c1e40c83697755640f138c </div>
<div style="display:none"> Gerrit-Change-Number: 7437 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Richard Mudgett <rmudgett@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Joshua Colp <jcolp@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Kevin Harwell <kharwell@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Sean Bright <sean.bright@gmail.com> </div>