fixeria submitted this change.

View Change


Approvals: osmith: Looks good to me, but someone else must approve Jenkins Builder: Verified pespin: Looks good to me, approved
{cbc,remsim}: fix unbound 'use_ssl' field in HTTP_Adapter_Params

A recent commit added a new field to record HTTP_Adapter_Params, but
forgot to update the callers f_http_init() to actually set this field.

Change-Id: I534d087b71cefd683a0c7411a415049c5e1ee519
Fixes: 832b1efe "HTTP_Adapter: Allow API users to enable/disable SSL"
---
M cbc/CBC_Tests.ttcn
M remsim/RemsimServer_Tests.ttcn
2 files changed, 18 insertions(+), 4 deletions(-)

diff --git a/cbc/CBC_Tests.ttcn b/cbc/CBC_Tests.ttcn
index 57f4d9e..52f6378 100644
--- a/cbc/CBC_Tests.ttcn
+++ b/cbc/CBC_Tests.ttcn
@@ -187,10 +187,10 @@
}

private function f_init(integer num_bsc := 0, integer num_mme := 0) runs on test_CT {
- var HTTP_Adapter_Params http_adapter_pars;
- http_adapter_pars := {
+ var HTTP_Adapter_Params http_adapter_pars := {
http_host := mp_cbc_host,
- http_port := mp_cbc_ecbe_port
+ http_port := mp_cbc_ecbe_port,
+ use_ssl := false
};
f_http_init(http_adapter_pars);

diff --git a/remsim/RemsimServer_Tests.ttcn b/remsim/RemsimServer_Tests.ttcn
index 94529b2..23ea8cb 100644
--- a/remsim/RemsimServer_Tests.ttcn
+++ b/remsim/RemsimServer_Tests.ttcn
@@ -71,7 +71,8 @@
var HTTP_Adapter_Params http_adapter_pars;
http_adapter_pars := {
http_host := mp_server_ip,
- http_port := mp_rsres_port
+ http_port := mp_rsres_port,
+ use_ssl := false
};
f_http_init(http_adapter_pars);
f_rsres_post_reset();

To view, visit change 36636. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I534d087b71cefd683a0c7411a415049c5e1ee519
Gerrit-Change-Number: 36636
Gerrit-PatchSet: 2
Gerrit-Owner: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged