<p>Joshua Colp <strong>merged</strong> this change.</p><p><a href="https://gerrit.asterisk.org/7008">View Change</a></p><div style="white-space:pre-wrap">Approvals:
Joshua Colp: Looks good to me, but someone else must approve; Approved for Submit
Richard Mudgett: Looks good to me, approved
</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/7008">change 7008</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/7008"/><meta itemprop="name" content="View Change"/></div></div>
<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: certified/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: 7008 </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: Joshua Colp <jcolp@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Richard Mudgett <rmudgett@digium.com> </div>