<p>Richard Mudgett has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.asterisk.org/7900">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">res_pjsip_endpoint_identifier_ip.c: Remove unnecessary requirement.<br><br>The requirement that "ip" must be in the endpoint identify_by list to<br>allow the type=identify method to identify the endpoint is not necessary.<br>The "ip" identifier method can match one and only one endpoint.  To even<br>work, the "ip" identifier method configuration must explicitly specify the<br>identified endpoint.  Therefore, why bother configuring the type=identify<br>identifier in the first place?  The requirement only adds the potential<br>for configuration errors for no benefit.  Even worse, those configuration<br>errors cannot be detected when the configuration loads.  The requirement<br>was introduced with the ASTERISK_27206 patch.<br><br>* Remove the code change that enforces the requiremnt.  Listing the "ip"<br>method in the identify_by value is simply documentation.<br><br>Change-Id: Ia057f92a33fb5d9f51dc5d5692e3d5ee1a6f2c11<br>---<br>M res/res_pjsip_endpoint_identifier_ip.c<br>1 file changed, 0 insertions(+), 7 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/00/7900/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/res/res_pjsip_endpoint_identifier_ip.c b/res/res_pjsip_endpoint_identifier_ip.c<br>index e40f9bf..7cc2dc5 100644<br>--- a/res/res_pjsip_endpoint_identifier_ip.c<br>+++ b/res/res_pjsip_endpoint_identifier_ip.c<br>@@ -229,14 +229,7 @@<br>         }<br> <br>  endpoint = ast_sorcery_retrieve_by_id(ast_sip_get_sorcery(), "endpoint", match->endpoint_name);<br>-<br>       if (endpoint) {<br>-              if (!(endpoint->ident_method & AST_SIP_ENDPOINT_IDENTIFY_BY_IP)) {<br>-                    ast_debug(3, "Endpoint '%s' found for '%s' but 'ip' method not supported'\n", match->endpoint_name,<br>-                         ast_sockaddr_stringify(&addr));<br>-                  ao2_cleanup(endpoint);<br>-                       return NULL;<br>-         }<br>             ast_debug(3, "Retrieved endpoint %s\n", ast_sorcery_object_get_id(endpoint));<br>       } else {<br>              ast_log(LOG_WARNING, "Identify section '%s' points to endpoint '%s' but endpoint could not be looked up\n",<br></pre><p>To view, visit <a href="https://gerrit.asterisk.org/7900">change 7900</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/7900"/><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-MessageType: newchange </div>
<div style="display:none"> Gerrit-Change-Id: Ia057f92a33fb5d9f51dc5d5692e3d5ee1a6f2c11 </div>
<div style="display:none"> Gerrit-Change-Number: 7900 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Richard Mudgett <rmudgett@digium.com> </div>