<p> Attention is currently required from: Alexei Gradinari. </p>
<p><a href="https://gerrit.asterisk.org/c/asterisk/+/18892">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_pubsub.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/+/18892/comment/014c6cec_9b3a49e8">Patch Set #1, Line 3033:</a> <code style="font-family:monospace,monospace">    if (!PJSIP_URI_SCHEME_IS_SIP(request_uri) && !PJSIP_URI_SCHEME_IS_SIPS(request_uri)) {</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">why don't use ast_sip_is_allowed_uri?</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">We've decided to only make changes for basic calls at the moment due to the reports of tel URI usage being primarily limited to providers, which would not use pubsub. Some areas have been prepped for it, but this still needs to be SIP or SIPS for now. There are some parts of the code that have pjsip_sip_uri as the data type (functions and even structs) that makes it very difficult to make changes without having to break the API. It's not completely out of the question though, and if someone in the community gets around to it before we do, that's fine as well!</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/18892">change 18892</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/+/18892"/><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: If5729e6cd583be7acf666373bf9f1b9d653ec29a </div>
<div style="display:none"> Gerrit-Change-Number: 18892 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Benjamin Keith Ford <bford@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-CC: Alexei Gradinari <alex2grad@gmail.com> </div>
<div style="display:none"> Gerrit-Attention: Alexei Gradinari <alex2grad@gmail.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 09 Aug 2022 13:05:02 +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: Alexei Gradinari <alex2grad@gmail.com> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>