<p> Attention is currently required from: N A, Joshua Colp, George Joseph. </p>
<p><a href="https://gerrit.asterisk.org/c/asterisk/+/18830">View Change</a></p><p>2 comments:</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/+/18830/comment/b693df0e_07ffef1c">Patch Set #1, Line 1703:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><pre style="font-family: monospace,monospace; white-space: pre-wrap;">     /* Set URI (inside the <>, as opposed to header (outside the <>)) parameters, if requested.<br>        * This is often necessary for SS7 trunking over SIP. Most carriers require<br>    * that added parameters be URI parameters (as opposed to header parameters).<br>  *<br>     * other_param is how to add custom URI parameters to the header<br>       * XXX: header_param is for header parameters, but doesn't seem to work at the moment.<br>     *<br>     * Especially when dealing with SS7 or TDM elements over SIP,<br>  * injecting parameters into the user and/or method parameters<br>         * is required.<br>        *<br>     * For example, if wanted to set the isup-oli parameter to the channel's ANI2 value:<br>       * e.g. pjsip_param_add(dlg_pool, &dlg_info_uri->other_param, "isup-oli", ani2);<br></pre></blockquote></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">I can trim this down as suggested. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;"><sip:foo@bar.com;p1=abc;p2=def?h1=qrs&h2=tuv>;o1=foo;o2=bar</p><p style="white-space: pre-wrap; word-wrap: break-word;">p1 and p2 are URI parameters<br>h1 and h2 are headers ("URI headers" to avoid confusion)<br>o1 and o2 are parameters and depending on the header they appear in (From, To, Contact, etc.) could be called "from parameters" or just "header parameters."</p><p style="white-space: pre-wrap; word-wrap: break-word;">The important takeaway is that o1 and o2 have nothing to do with the URI.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/18830/comment/c70d555a_9a01e659">Patch Set #1, Line 1731:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><pre style="font-family: monospace,monospace; white-space: pre-wrap;">    while ((param_value = strsep(&params, ","))) {<br>                  param_name = strsep(&param_value, "=");<br></pre></blockquote></p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Parameter names wouldn't contain equal signs, so I don't think the strsep on = is an issue.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">They are permitted to contain them as long as they are escaped. We are leveraging the PJSIP stack to do that escaping for us (ideally) so we should be able to add a parameter with name "name" and a value of "surname=bright,first=sean" and it should work fine.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/18830">change 18830</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/+/18830"/><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: Ifb1bc3c512ad5f6faeaebd7817f004a2ecbd6428 </div>
<div style="display:none"> Gerrit-Change-Number: 18830 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: N A <mail@interlinked.x10host.com> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Reviewer: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Joshua Colp <jcolp@sangoma.com> </div>
<div style="display:none"> Gerrit-Reviewer: Sean Bright <sean@seanbright.com> </div>
<div style="display:none"> Gerrit-Attention: N A <mail@interlinked.x10host.com> </div>
<div style="display:none"> Gerrit-Attention: Joshua Colp <jcolp@sangoma.com> </div>
<div style="display:none"> Gerrit-Attention: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 25 Jul 2022 16:35:15 +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: Sean Bright <sean@seanbright.com> </div>
<div style="display:none"> Comment-In-Reply-To: N A <mail@interlinked.x10host.com> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>