[Asterisk-code-review] testsuite: SIP route header is missing on UPDATE (testsuite[19])

Friendly Automation asteriskteam at digium.com
Mon May 2 10:01:40 CDT 2022


Friendly Automation has submitted this change. ( https://gerrit.asterisk.org/c/testsuite/+/18488 )

Change subject: testsuite: SIP route header is missing on UPDATE
......................................................................

testsuite: SIP route header is missing on UPDATE

Fix escape characters on variable assignment

ASTERISK-29955

Change-Id: I2cbce7806007f28aeb67feebe17c11c437307359
---
M tests/channels/SIP/sip_semi_attended_transfer_record_route/sipp/uac-no-hangup.xml
M tests/channels/SIP/sip_semi_attended_transfer_record_route/sipp/uas.xml
M tests/channels/pjsip/transfers/attended_transfer/nominal/callee_local_semi_attended_transfer_record_route/sipp/uac-no-hangup.xml
M tests/channels/pjsip/transfers/attended_transfer/nominal/callee_local_semi_attended_transfer_record_route/sipp/uas.xml
4 files changed, 4 insertions(+), 4 deletions(-)

Approvals:
  Kevin Harwell: Looks good to me, but someone else must approve
  George Joseph: Looks good to me, approved
  Friendly Automation: Approved for Submit



diff --git a/tests/channels/SIP/sip_semi_attended_transfer_record_route/sipp/uac-no-hangup.xml b/tests/channels/SIP/sip_semi_attended_transfer_record_route/sipp/uac-no-hangup.xml
index d678884..8711da7 100644
--- a/tests/channels/SIP/sip_semi_attended_transfer_record_route/sipp/uac-no-hangup.xml
+++ b/tests/channels/SIP/sip_semi_attended_transfer_record_route/sipp/uac-no-hangup.xml
@@ -7,7 +7,7 @@
 		<action>
 			<assignstr
 				assign_to="rr_out"
-				value="<sip:[local_ip]:[local_port];transport=[transport];lr>" />
+				value="\x3Csip:[local_ip]:[local_port];transport=[transport];lr\x3E" />
 		</action>
 	</nop>
 
diff --git a/tests/channels/SIP/sip_semi_attended_transfer_record_route/sipp/uas.xml b/tests/channels/SIP/sip_semi_attended_transfer_record_route/sipp/uas.xml
index 4899731..91eb681 100644
--- a/tests/channels/SIP/sip_semi_attended_transfer_record_route/sipp/uas.xml
+++ b/tests/channels/SIP/sip_semi_attended_transfer_record_route/sipp/uas.xml
@@ -25,7 +25,7 @@
 		<action>
 			<assignstr
 				assign_to="rr_out"
-				value="<sip:[local_ip]:[local_port];transport=[transport];lr>" />
+				value="\x3Csip:[local_ip]:[local_port];transport=[transport];lr\x3E" />
 		</action>
 	</nop>
 
diff --git a/tests/channels/pjsip/transfers/attended_transfer/nominal/callee_local_semi_attended_transfer_record_route/sipp/uac-no-hangup.xml b/tests/channels/pjsip/transfers/attended_transfer/nominal/callee_local_semi_attended_transfer_record_route/sipp/uac-no-hangup.xml
index dc8b57e..c841ed8 100644
--- a/tests/channels/pjsip/transfers/attended_transfer/nominal/callee_local_semi_attended_transfer_record_route/sipp/uac-no-hangup.xml
+++ b/tests/channels/pjsip/transfers/attended_transfer/nominal/callee_local_semi_attended_transfer_record_route/sipp/uac-no-hangup.xml
@@ -7,7 +7,7 @@
 		<action>
 			<assignstr
 				assign_to="rr_out"
-				value="<sip:[local_ip]:[local_port];transport=[transport];lr>" />
+				value="\x3Csip:[local_ip]:[local_port];transport=[transport];lr\x3E" />
 		</action>
 	</nop>
 
diff --git a/tests/channels/pjsip/transfers/attended_transfer/nominal/callee_local_semi_attended_transfer_record_route/sipp/uas.xml b/tests/channels/pjsip/transfers/attended_transfer/nominal/callee_local_semi_attended_transfer_record_route/sipp/uas.xml
index 54fc4b9..74eb121 100644
--- a/tests/channels/pjsip/transfers/attended_transfer/nominal/callee_local_semi_attended_transfer_record_route/sipp/uas.xml
+++ b/tests/channels/pjsip/transfers/attended_transfer/nominal/callee_local_semi_attended_transfer_record_route/sipp/uas.xml
@@ -25,7 +25,7 @@
 		<action>
 			<assignstr
 				assign_to="rr_out"
-				value="<sip:[local_ip]:[local_port];transport=[transport];lr>" />
+				value="\x3Csip:[local_ip]:[local_port];transport=[transport];lr\x3E" />
 		</action>
 	</nop>
 

-- 
To view, visit https://gerrit.asterisk.org/c/testsuite/+/18488
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: testsuite
Gerrit-Branch: 19
Gerrit-Change-Id: I2cbce7806007f28aeb67feebe17c11c437307359
Gerrit-Change-Number: 18488
Gerrit-PatchSet: 2
Gerrit-Owner: Michael Bradeen <mbradeen at sangoma.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Kevin Harwell <kharwell at digium.com>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20220502/368588a1/attachment.html>


More information about the asterisk-code-review mailing list