<p>Richard Mudgett <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/7937">View Change</a></p><p>Patch set 1:</p><p style="white-space: pre-wrap; word-wrap: break-word;">I'll remove the unneeded sections of the normal.xml files Monday.</p><p>(2 comments)</p><ul style="list-style: none; padding-left: 20px;"><li><p><a href="https://gerrit.asterisk.org/#/c/7937/1/tests/channels/pjsip/identify/header_ordering_header_ip/configs/ast1/pjsip.conf">File tests/channels/pjsip/identify/header_ordering_header_ip/configs/ast1/pjsip.conf:</a></p><ul style="list-style: none; padding-left: 20px;"><li><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/7937/1/tests/channels/pjsip/identify/header_ordering_header_ip/configs/ast1/pjsip.conf@3">Patch Set #1, Line 3:</a> <code style="font-family:monospace,monospace">header,ip</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">This should be reversed.  Since this is the header test, you also want to m</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">You have it backwards.  This is the header test so we need to match the incoming call to the header endpoint.</p><p style="white-space: pre-wrap; word-wrap: break-word;">The incoming call request message matches both ip and header identify sections.  Since header is listed *first* in endpoint_identifier_order then the call is matched to the header endpoint first.</p><p style="white-space: pre-wrap; word-wrap: break-word;">The other test is setup to match the other way for the same incoming call request message.</p></li></ul></li><li><p><a href="https://gerrit.asterisk.org/#/c/7937/1/tests/channels/pjsip/identify/header_ordering_ip_header/configs/ast1/pjsip.conf">File tests/channels/pjsip/identify/header_ordering_ip_header/configs/ast1/pjsip.conf:</a></p><ul style="list-style: none; padding-left: 20px;"><li><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/7937/1/tests/channels/pjsip/identify/header_ordering_ip_header/configs/ast1/pjsip.conf@3">Patch Set #1, Line 3:</a> <code style="font-family:monospace,monospace">ip,header</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">This should be reversed.  Since this is the ip test, you also want to make </blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">You have it backwards.  This is the ip test so we need to match the incoming call to the ip endpoint.</p><p style="white-space: pre-wrap; word-wrap: break-word;">The incoming call request message matches both ip and header identify sections.  Since ip is listed *first* in endpoint_identifier_order then the call is matched to the ip endpoint first.</p><p style="white-space: pre-wrap; word-wrap: break-word;">The other test is setup to match the other way for the same incoming call request message.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/7937">change 7937</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/7937"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: testsuite </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I12560bdcc36f7df170de79e11666fe6b812e67f4 </div>
<div style="display:none"> Gerrit-Change-Number: 7937 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Richard Mudgett <rmudgett@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Richard Mudgett <rmudgett@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 12 Jan 2018 16:25:46 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>