<p> Attention is currently required from: Joshua Colp. </p>
<p><a href="https://gerrit.asterisk.org/c/asterisk/+/18371">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="null">File res/res_pjsip_session.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/18371/comment/71ebfdfc_b2d5ef18">Patch Set #1, Line 3425:</a> <code style="font-family:monospace,monospace">                   if (ast_stream_get_state(req_stream) == AST_STREAM_STATE_SENDONLY) {</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Gerrit doesn't work using tiny commits and separate reviews - you amend and update this review. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I see what you're saying and the bridge definitely seems like the right place for this reversal of stream direction to occur. However, the bridge code does not yet come into play when the outgoing stream of the second call leg is initially created. That is why the SDP negotiation results in an incorrect stream state when the call is being answered.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Here is an example of what seems to be happening:</p><p style="white-space: pre-wrap; word-wrap: break-word;">1. Caller sends sendonly -> local_stream of call leg 1 is correctly set recvonly<br>2. Outgoing stream is created and clones local topology of call leg 1 -> local_stream of call leg 2 is recvonly as well (should be sendonly)<br>3. Callee answers with recvonly -> SDP negotiation of 2nd call leg receives recvonly for both local_sdp and remote_sdp -> no media is transmitted</p><p style="white-space: pre-wrap; word-wrap: break-word;">So it sounds like the more sensible place for the stream reversal to occur would be further up the chain of execution in app_dial.c:dial_exec_full() after the topology of the first channel is cloned (which I have tried and it fixes ASTERISK-29655 as well).</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/18371">change 18371</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/+/18371"/><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: I294dc834ac9a5f048b101b691669959e9df630e1 </div>
<div style="display:none"> Gerrit-Change-Number: 18371 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Maximilian Fridrich <m.fridrich@commend.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-CC: Florian Floimair <f.floimair@commend.com> </div>
<div style="display:none"> Gerrit-Attention: Joshua Colp <jcolp@sangoma.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 14 Apr 2022 09:28:34 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Comment-In-Reply-To: Joshua Colp <jcolp@sangoma.com> </div>
<div style="display:none"> Comment-In-Reply-To: Maximilian Fridrich <m.fridrich@commend.com> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>