<p>Jenkins2 <strong>merged</strong> this change.</p><p><a href="https://gerrit.asterisk.org/7154">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  Kevin Harwell: Looks good to me, approved
  Jenkins2: Approved for Submit

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">res_pjsip: Add to list of valid characters for from_user.<br><br>Fixes a regression where some characters were unable to be used in<br>the from_user field of an endpoint. Additionally, the backtick was<br>removed from the list of valid characters, since it is not valid,<br>and it was replaced with a single quote, which is a valid character.<br><br>ASTERISK-27387<br><br>Change-Id: Id80c10a644508365c87b3182e99ea49da11b0281<br>(cherry picked from commit ffcb7e2a2540181ea41062ca0e1bc3e4fed9b3a5)<br>---<br>M res/res_pjsip/pjsip_configuration.c<br>1 file changed, 2 insertions(+), 2 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/res/res_pjsip/pjsip_configuration.c b/res/res_pjsip/pjsip_configuration.c<br>index 8106676..fa54448 100644<br>--- a/res/res_pjsip/pjsip_configuration.c<br>+++ b/res/res_pjsip/pjsip_configuration.c<br>@@ -1137,11 +1137,11 @@<br> {<br>     struct ast_sip_endpoint *endpoint = obj;<br>      /* Valid non-alphanumeric characters for URI */<br>-      char *valid_uri_marks = "-_.!~*`()";<br>+       char *valid_uri_marks = "-._~%!$&'()*+,;=:";<br>        const char *val;<br> <br>   for (val = var->value; *val; val++) {<br>-             if (!strchr(valid_uri_marks, *val) && !isdigit(*val) && !isalpha(*val)) {<br>+            if (!isalpha(*val) && !isdigit(*val) && !strchr(valid_uri_marks, *val)) {<br>                     ast_log(LOG_ERROR, "Error configuring endpoint '%s' - '%s' field "<br>                  "contains invalid character '%c'\n",<br>                        ast_sorcery_object_get_id(endpoint), var->name, *val);<br></pre><p>To view, visit <a href="https://gerrit.asterisk.org/7154">change 7154</a>. To unsubscribe, 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/7154"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 13.18 </div>
<div style="display:none"> Gerrit-MessageType: merged </div>
<div style="display:none"> Gerrit-Change-Id: Id80c10a644508365c87b3182e99ea49da11b0281 </div>
<div style="display:none"> Gerrit-Change-Number: 7154 </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: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Kevin Harwell <kharwell@digium.com> </div>