<p> Attention is currently required from: Michael Kuron. </p>
<p><a href="https://gerrit.asterisk.org/c/asterisk/+/19460">View Change</a></p><p>3 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/+/19460?tab=comments">Patch Set #2:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Is there a way to register new endpoint settings without having to modify chan_pjsip? […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">There is not. It has to be modified, and an Alembic script created[1] to cover the realtime aspect. A sample config file is in the tree in contrib/ast-db-manage which can be provided to alembic using the -c option:</p><p style="white-space: pre-wrap; word-wrap: break-word;">alembic -c config.ini.sample revision -m "add aoc option"</p><p style="white-space: pre-wrap; word-wrap: break-word;">As for test coverage, the testsuite is a separate repo[2] and there are tests in tests/channels/pjsip for PJSIP. Finding one that is closest to what you want is the easiest option and then copying it. Instructions for setting up the testsuite and running it are on the wiki[3].</p><p style="white-space: pre-wrap; word-wrap: break-word;">[1] https://alembic.sqlalchemy.org/en/latest/tutorial.html#create-a-migration-script<br>[2] https://github.com/asterisk/testsuite<br>[3] https://wiki.asterisk.org/wiki/display/AST/Using+Python3</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File channels/chan_pjsip.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/+/19460/comment/bbf431ad_2a43f0ad">Patch Set #2, Line 1834:</a> <code style="font-family:monospace,monospace">    case AST_CONTROL_AOC:</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">I am using a framehook now and it works fine, but I get these warning messages: […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Can the framehook return an &ast_null_frame instead as the frame has been handled by it?</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File res/res_pjsip_aoc.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/+/19460/comment/6767ff6b_f3ba1a70">Patch Set #2, Line 491:</a> <code style="font-family:monospace,monospace">static pj_status_t aoc_outgoing_response(pjsip_tx_data *tdata)</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">In my testing, aoc_bye_supplement.outgoing_response never got triggered. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">There's a limited number of pjsip_modules that can exist at any time, so trying to avoid them is best. I think investigating why in the particular use case it wasn't called would be good.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/19460">change 19460</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/+/19460"/><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-Change-Id: Iebb7ad0d5f88526bc6629d3a1f9f11665434d333 </div>
<div style="display:none"> Gerrit-Change-Number: 19460 </div>
<div style="display:none"> Gerrit-PatchSet: 4 </div>
<div style="display:none"> Gerrit-Owner: Michael Kuron <m.kuron@gmx.de> </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-Attention: Michael Kuron <m.kuron@gmx.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Sat, 29 Oct 2022 18:37:21 +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: Michael Kuron <m.kuron@gmx.de> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>