<p> Attention is currently required from: Joshua Colp, George Joseph. </p>
<p><a href="https://gerrit.asterisk.org/c/asterisk/+/18830">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">Patchset:</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/+/18830?tab=comments">Patch Set #1:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">What does the pjproject library have to do with a testsuite test? The test should just use a SIPp sc […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I'm using Asterisk, not SIPp, since that makes more sense for this kind of test. In an ideal end to end test, you set the ANI2 manually on an outbound call and then on an incoming call that value should be present on CALLERID(ani2), via a loopback PJSIP call. That effectively tests 3 things at once.</p><p style="white-space: pre-wrap; word-wrap: break-word;">This change itself works (it's on the outbound side). It's easy to confirm by doing a SIP debug and making some outbound calls. I couldn't get header parameters working due to pjproject issues so that's not part of this patch, just URI parameters.</p><p style="white-space: pre-wrap; word-wrap: break-word;">The problems with the test are really on the incoming side. It seems like perhaps the parameters searched are either only header or URI parameters, but one of them is completely missing and can't be retrieved. So pjsip_param_find can't be used to confirm the parameters.</p><p style="white-space: pre-wrap; word-wrap: break-word;">It *IS* in the incoming invite (again, this can be confirmed using a debug) and if we wanted to just make the test work, a quick and dirty way would be just using PJSIP_HEADER and confirm that the substring exists using a regex. Not as elegant, but probably more reliable at this point since pjproject is so flaky.</p><p style="white-space: pre-wrap; word-wrap: break-word;">However, aside from using that for the test or not, it's a problem in and of itself if pjproject isn't cooperating for that, so this is a separate thing that will need to be looked into.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Maybe I can just do the PJSIP_HEADER/REGEX method for now, and look into this on the side since it's a really a separate issue pertaining to how pjproject parses things on incoming calls, and the structures in play.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/18830">change 18830</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/+/18830"/><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-Change-Id: Ifb1bc3c512ad5f6faeaebd7817f004a2ecbd6428 </div>
<div style="display:none"> Gerrit-Change-Number: 18830 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: N A <mail@interlinked.x10host.com> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Reviewer: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Joshua Colp <jcolp@sangoma.com> </div>
<div style="display:none"> Gerrit-Attention: Joshua Colp <jcolp@sangoma.com> </div>
<div style="display:none"> Gerrit-Attention: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Sat, 23 Jul 2022 00:00:49 +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: Joshua Colp <jcolp@sangoma.com> </div>
<div style="display:none"> Comment-In-Reply-To: N A <mail@interlinked.x10host.com> </div>
<div style="display:none"> Comment-In-Reply-To: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>