<p>Joshua Colp <strong>uploaded patch set #2</strong> to this change.</p><p><a href="https://gerrit.asterisk.org/c/asterisk/+/13735">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">res_pjsip_session: Fix off-nominal session refreshes.<br><br>Given a scenario where session refreshes occur close to<br>each other while another is finishing it was possible for<br>the session refreshes to occur out of order. It was<br>also possible for session refreshes to be delayed for<br>quite some time if a session refresh did not result in<br>a topology change.<br><br>For the out of order session refreshes the first session<br>refresh would be queued due to a transaction in progress.<br>This transaction would then finish. When finished a<br>separate task to process the delayed requests queue<br>would be queued for handling. A second refresh would<br>be requested internally before this delayed request<br>queued task was processed. As no transaction was in<br>progress this session refresh would be immediately<br>handled before the queued session refresh.<br><br>The code will now check if any delayed requests exist<br>before allowing a session refresh to immediately occur.<br>If any exist then the session refresh is queued.<br><br>For the delayed session refreshes if a session refresh<br>did not result in a topology change the attempt would<br>be immediately stopped and no other delayed requests would<br>be processed.<br><br>The code will now go through the entire delayed requests<br>queue until a delayed request results in a request<br>actually being sent.<br><br>ASTERISK-28730<br><br>Change-Id: Ied640280133871f77d3f332be62265e754605088<br>---<br>M res/res_pjsip_session.c<br>1 file changed, 62 insertions(+), 11 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/35/13735/2</pre><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/13735">change 13735</a>. To unsubscribe, or for help writing mail filters, 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/c/asterisk/+/13735"/><meta itemprop="name" content="View Change"/></div></div>
<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 16 </div>
<div style="display:none"> Gerrit-Change-Id: Ied640280133871f77d3f332be62265e754605088 </div>
<div style="display:none"> Gerrit-Change-Number: 13735 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Joshua Colp <jcolp@sangoma.com> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Reviewer: Joshua Colp <jcolp@sangoma.com> </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>