<p> Attention is currently required from: N A, Henning Westerholt. </p>
<p>Patch set 3:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ffd4d4; color: #000000;">Code-Review -1</span></p><p><a href="https://gerrit.asterisk.org/c/asterisk/+/18990">View Change</a></p><p>4 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="null">Patchset:</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/+/18990?tab=comments">Patch Set #3:</a> </p><p style="white-space: pre-wrap; word-wrap: break-word;">This also needs an Alembic migration script[1] since this adds a new option. There is a sample config script in contrib/ast-db-manage which can be passed to alembic using the -c option. You can look through past migration scripts to see how the Python is done.</p><p style="white-space: pre-wrap; word-wrap: break-word;">[1] https://alembic.sqlalchemy.org/en/latest/tutorial.html#create-a-migration-script</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File configs/samples/pjsip.conf.sample:</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/+/18990/comment/1e9206dc_5fcd6241">Patch Set #3, Line 1281:</a> <code style="font-family:monospace,monospace">                    ; Return all codecs on empty re-INVITE.</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">This needs to be more thoroughly documented about what it actually does. For example:</p><p style="white-space: pre-wrap; word-wrap: break-word;">On reception of a re-INVITE without SDP Asterisk will send an SDP offer in the 200 OK response containing all configured codecs on the endpoint, instead of simply those that have already been negotiated.</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File res/res_pjsip/pjsip_config.xml:</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/+/18990/comment/c4171d0b_ff90ad68">Patch Set #3, Line 2077:</a> <code style="font-family:monospace,monospace">                                             Return all codecs on empty re-INVITE. RFC 3261</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Comment I provided per sample config also applies here</p></li></ul></li><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/+/18990/comment/8207606f_ae601cb3">Patch Set #3, Line 5067:</a> <code style="font-family:monospace,monospace">            if (session->active_media_state->topology && !answer_all_codecs) {</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">If there are multiple streams, what is the consequence of not using the active media state topology? What is the resulting SDP like? Should the variable name instead reflect what it is actually doing, that is: ignoring active stream topology</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/18990">change 18990</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/+/18990"/><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: I69763708d5039d512f391e296ee8a4d43a1e2148 </div>
<div style="display:none"> Gerrit-Change-Number: 18990 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: Henning Westerholt <hw@gilawa.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-Reviewer: N A <mail@interlinked.x10host.com> </div>
<div style="display:none"> Gerrit-Attention: N A <mail@interlinked.x10host.com> </div>
<div style="display:none"> Gerrit-Attention: Henning Westerholt <hw@gilawa.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 29 Aug 2022 11:06:22 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: Yes </div>
<div style="display:none"> Gerrit-MessageType: comment </div>