October 2025 - gerrit-log - lists.osmocom.org

[S] Change in osmo-ttcn3-hacks[master]: smdpp_Tests: rename smdp_server_url to smdp_server_fqdn
by dexter 21 Oct '25

21 Oct '25

dexter has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41233?usp=email ) Change subject: smdpp_Tests: rename smdp_server_url to smdp_server_fqdn ...................................................................... smdpp_Tests: rename smdp_server_url to smdp_server_fqdn The member name "smdp_server_url" is technically not correct as the content is an FQDN. Let's rename it to "smdp_server_fqdn". Let's also put a comment in which we explain why FQDN is hardcoded and not available as configurable module parameter. Related: SYS#7339 Change-Id: I0224c838a368656821fff6014b0df7eb49466b0a --- M smdpp/smdpp_Tests.ttcn 1 file changed, 13 insertions(+), 10 deletions(-) Approvals: Jenkins Builder: Verified osmith: Looks good to me, approved diff --git a/smdpp/smdpp_Tests.ttcn b/smdpp/smdpp_Tests.ttcn index 8ba125a..548b33a 100644 --- a/smdpp/smdpp_Tests.ttcn +++ b/smdpp/smdpp_Tests.ttcn @@ -360,7 +360,7 @@ } type record smdpp_ConnHdlrPars { - charstring smdp_server_url, + charstring smdp_server_fqdn, integer smdp_es9p_server_port, integer smdp_es2p_server_port, charstring cert_path, @@ -381,7 +381,10 @@ private function f_init_pars() runs on MTC_CT return smdpp_ConnHdlrPars { var smdpp_ConnHdlrPars pars := { - smdp_server_url := "testsmdpplus1.example.com", + /* Unfortunately the SMDP server FQDN cannot be made configurable via a module parameter since it always + * to match the FQDN in the test SSL certificates. To resolve the server FQDN to an IP address, an entry + * in /etc/hosts is sufficient. */ + smdp_server_fqdn := "testsmdpplus1.example.com", smdp_es9p_server_port := 8000, smdp_es2p_server_port := 8000, cert_path := "./sgp26/CertificateIssuer", @@ -444,7 +447,7 @@ ext_logInfo("Initializing RSP client"); g_rsp_client_handle := ext_RSPClient_create( - g_pars_smdpp.smdp_server_url, + g_pars_smdpp.smdp_server_fqdn, g_pars_smdpp.smdp_es9p_server_port, g_pars_smdpp.cert_path, g_pars_smdpp.cert_name_filter @@ -675,7 +678,7 @@ var RemoteProfileProvisioningRequest request := { initiateAuthenticationRequest := { euiccChallenge := g_euiccChallenge, - smdpAddress := g_pars_smdpp.smdp_server_url, + smdpAddress := g_pars_smdpp.smdp_server_fqdn, euiccInfo1 := euiccInfo1 } }; @@ -1204,7 +1207,7 @@ ) runs on smdpp_ConnHdlr return EuiccSigned1 { return { transactionId := g_transactionId, - serverAddress := g_pars_smdpp.smdp_server_url, + serverAddress := g_pars_smdpp.smdp_server_fqdn, serverChallenge := g_serverChallenge, euiccInfo2 := euiccInfo2, ctxParams1 := ctxParams @@ -1540,7 +1543,7 @@ var NotificationMetadata notifMeta := { seqNumber := 1, profileManagementOperation := c_notificationInstall, - notificationAddress := g_pars_smdpp.smdp_server_url, + notificationAddress := g_pars_smdpp.smdp_server_fqdn, iccid := omit }; @@ -1642,7 +1645,7 @@ var EuiccSigned1 euiccSigned1 := { transactionId := actualTransactionId, - serverAddress := g_pars_smdpp.smdp_server_url, + serverAddress := g_pars_smdpp.smdp_server_fqdn, serverChallenge := serverChallenge, euiccInfo2 := actualEuiccInfo2, ctxParams1 := ctxParams @@ -1993,7 +1996,7 @@ /* Build euiccSigned1 */ var EuiccSigned1 euiccSigned1 := { transactionId := transactionId, - serverAddress := g_pars_smdpp.smdp_server_url, + serverAddress := g_pars_smdpp.smdp_server_fqdn, serverChallenge := serverChallenge, euiccInfo2 := euiccInfo2, ctxParams1 := ctxParams @@ -3736,7 +3739,7 @@ var EuiccSigned1 euiccSig := { transactionId := g_transactionId, - serverAddress := g_pars_smdpp.smdp_server_url, + serverAddress := g_pars_smdpp.smdp_server_fqdn, serverChallenge := g_serverChallenge, euiccInfo2 := euic2, ctxParams1 := ctxpar @@ -4101,7 +4104,7 @@ var NotificationMetadata notifMeta := { seqNumber := 2, /* Different sequence number from PIR */ profileManagementOperation := c_notificationEnable, - notificationAddress := g_pars_smdpp.smdp_server_url, + notificationAddress := g_pars_smdpp.smdp_server_fqdn, iccid := omit /* No specific ICCID for this test */ }; -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41233?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: I0224c838a368656821fff6014b0df7eb49466b0a Gerrit-Change-Number: 41233 Gerrit-PatchSet: 1 Gerrit-Owner: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>

1 0

[S] Change in osmo-ttcn3-hacks[master]: smdpp_Tests: rename smdp_server_url to smdp_server_fqdn
by osmith 21 Oct '25

21 Oct '25

Attention is currently required from: dexter. osmith has posted comments on this change by dexter. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41233?usp=email ) Change subject: smdpp_Tests: rename smdp_server_url to smdp_server_fqdn ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41233?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: I0224c838a368656821fff6014b0df7eb49466b0a Gerrit-Change-Number: 41233 Gerrit-PatchSet: 1 Gerrit-Owner: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Attention: dexter <pmaier(a)sysmocom.de> Gerrit-Comment-Date: Tue, 21 Oct 2025 13:15:05 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[XS] Change in osmo-ttcn3-hacks[master]: smdpp_Tests: fix sourcecode formatting, remove line breaks
by dexter 21 Oct '25

21 Oct '25

dexter has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41232?usp=email ) Change subject: smdpp_Tests: fix sourcecode formatting, remove line breaks ...................................................................... smdpp_Tests: fix sourcecode formatting, remove line breaks Change-Id: I50175cd2989d6bfbe220c33e73a1724166ca548f Related: SYS#7339 --- M smdpp/smdpp_Tests.cfg M smdpp/smdpp_Tests.ttcn 2 files changed, 0 insertions(+), 3 deletions(-) Approvals: Jenkins Builder: Verified osmith: Looks good to me, approved diff --git a/smdpp/smdpp_Tests.cfg b/smdpp/smdpp_Tests.cfg index 32bab0d..35f3140 100644 --- a/smdpp/smdpp_Tests.cfg +++ b/smdpp/smdpp_Tests.cfg @@ -15,8 +15,6 @@ #*.FileMask:=LOG_ALL | MATCHING | DEBUG; *.ConsoleMask:=ERROR | TESTCASE | USER | VERDICTOP | WARNING ; - - [TESTPORT_PARAMETERS] system.HTTP_server_port.KEYFILE := "../ipad/example_ca/pki/private/alttest.key" system.HTTP_server_port.CERTIFICATEFILE := "../ipad/example_ca/pki/issued/alttest.crt" diff --git a/smdpp/smdpp_Tests.ttcn b/smdpp/smdpp_Tests.ttcn index 79af08c..8ba125a 100644 --- a/smdpp/smdpp_Tests.ttcn +++ b/smdpp/smdpp_Tests.ttcn @@ -28,7 +28,6 @@ import from PKIX1Implicit88_Templates all; import from PKIX1Implicit88_Types all; - import from es12_Types_JSON all; import from es9p_Types_JSON all; import from es2p_Types_JSON all; -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41232?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: I50175cd2989d6bfbe220c33e73a1724166ca548f Gerrit-Change-Number: 41232 Gerrit-PatchSet: 1 Gerrit-Owner: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>

1 0

[XS] Change in osmo-ttcn3-hacks[master]: smdpp_Tests: fix sourcecode formatting, remove line breaks
by osmith 21 Oct '25

21 Oct '25

Attention is currently required from: dexter. osmith has posted comments on this change by dexter. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41232?usp=email ) Change subject: smdpp_Tests: fix sourcecode formatting, remove line breaks ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41232?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: I50175cd2989d6bfbe220c33e73a1724166ca548f Gerrit-Change-Number: 41232 Gerrit-PatchSet: 1 Gerrit-Owner: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Attention: dexter <pmaier(a)sysmocom.de> Gerrit-Comment-Date: Tue, 21 Oct 2025 13:14:13 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[M] Change in osmo-ttcn3-hacks[master]: smdpp_Tests: simplify generation of smdpp_ConnHdlrPars (NIST vs. BRP)
by dexter 21 Oct '25

21 Oct '25

dexter has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41235?usp=email ) Change subject: smdpp_Tests: simplify generation of smdpp_ConnHdlrPars (NIST vs. BRP) ...................................................................... smdpp_Tests: simplify generation of smdpp_ConnHdlrPars (NIST vs. BRP) When we call f_init_pars() it will return a record with default parameters. However, those parameters are only a suitable default when the NIST curve is used. For the brainpool curve we will have to modify returned result. Let's simlify this by adding a parameter to f_init_pars() that allows us to switch between the defaults for NIST and BRP as needed. Realated: SYS#7339 Change-Id: Ied4ffa6ff4b3efbbcfc9204ce8f21ea7ca4becf5 --- M smdpp/smdpp_Tests.ttcn 1 file changed, 22 insertions(+), 40 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/35/41235/1 diff --git a/smdpp/smdpp_Tests.ttcn b/smdpp/smdpp_Tests.ttcn index 579f742..ddf094b 100644 --- a/smdpp/smdpp_Tests.ttcn +++ b/smdpp/smdpp_Tests.ttcn @@ -379,18 +379,14 @@ ES9EncodingMode es9_encoding_mode optional }; -private function f_init_pars() runs on MTC_CT return smdpp_ConnHdlrPars { +private function f_init_pars(boolean brainpool := false) runs on MTC_CT return smdpp_ConnHdlrPars { var smdpp_ConnHdlrPars pars := { /* Unfortunately the SMDP server FQDN cannot be made configurable via a module parameter since it always * to match the FQDN in the test SSL certificates. To resolve the server FQDN to an IP address, an entry * in /etc/hosts is sufficient. */ smdp_server_fqdn := "testsmdpplus1.example.com", - smdp_es9p_server_port := mp_es9plus_server_port_nist, smdp_es2p_server_port := mp_es2plus_server_port, cert_path := "./sgp26/CertificateIssuer", - cert_name_filter := "", // Load all certificates (both NIST and BRP) or "NIST" or "BRP" - euicc_cert_path := "./sgp26/eUICC/CERT_EUICC_ECDSA_NIST.der", - euicc_key_path := "./sgp26/eUICC/SK_EUICC_ECDSA_NIST.pem", confirmation_code := omit, // No confirmation code by default err_injection := omit, test_name := omit, @@ -402,6 +398,20 @@ metadata_segments := 1, es9_encoding_mode := omit /* Default to JSON mode */ }; + + /* GSMA SGP.22 specifies two different certificate curves, which require different default parameters */ + if (brainpool) { + pars.smdp_es9p_server_port := mp_es9plus_server_port_brp; + pars.cert_name_filter := "BRP"; + pars.euicc_cert_path := "./sgp26/eUICC/CERT_EUICC_ECDSA_BRP.der"; + pars.euicc_key_path := "./sgp26/eUICC/SK_EUICC_ECDSA_BRP.pem"; + } else { + pars.smdp_es9p_server_port := mp_es9plus_server_port_nist; + pars.cert_name_filter := "NIST"; + pars.euicc_cert_path := "./sgp26/eUICC/CERT_EUICC_ECDSA_NIST.der"; + pars.euicc_key_path := "./sgp26/eUICC/SK_EUICC_ECDSA_NIST.pem"; + } + return pars; } @@ -5570,11 +5580,7 @@ testcase TC_SM_DP_ES9_InitiateAuthenticationBRP_01_Nominal() runs on MTC_CT { - var smdpp_ConnHdlrPars pars := f_init_pars(); - pars.smdp_es9p_server_port := mp_es9plus_server_port_brp; /* BRP server port */ - pars.cert_name_filter := "BRP"; /* Use Brainpool certificates */ - pars.euicc_cert_path := "./sgp26/eUICC/CERT_EUICC_ECDSA_BRP.der"; - pars.euicc_key_path := "./sgp26/eUICC/SK_EUICC_ECDSA_BRP.pem"; + var smdpp_ConnHdlrPars pars := f_init_pars(brainpool := true); var smdpp_ConnHdlr vc_conn; f_init(testcasename()); vc_conn := f_start_handler(refers(f_TC_InitiateAuth_01_Nominal), pars); @@ -5583,11 +5589,7 @@ } testcase TC_SM_DP_ES9_AuthenticateClientBRP_01_Nominal() runs on MTC_CT { - var smdpp_ConnHdlrPars pars := f_init_pars(); - pars.smdp_es9p_server_port := mp_es9plus_server_port_brp; /* BRP server port */ - pars.cert_name_filter := "BRP"; /* Use Brainpool certificates */ - pars.euicc_cert_path := "./sgp26/eUICC/CERT_EUICC_ECDSA_BRP.der"; - pars.euicc_key_path := "./sgp26/eUICC/SK_EUICC_ECDSA_BRP.pem"; + var smdpp_ConnHdlrPars pars := f_init_pars(brainpool := true); var smdpp_ConnHdlr vc_conn; f_init(testcasename()); vc_conn := f_start_handler(refers(f_TC_AuthenticateClient_01_Nominal), pars); @@ -5596,11 +5598,7 @@ } testcase TC_SM_DP_ES9_GetBoundProfilePackageBRP_01_Nominal() runs on MTC_CT { - var smdpp_ConnHdlrPars pars := f_init_pars(); - pars.smdp_es9p_server_port := mp_es9plus_server_port_brp; /* BRP server port */ - pars.cert_name_filter := "BRP"; /* Use Brainpool certificates */ - pars.euicc_cert_path := "./sgp26/eUICC/CERT_EUICC_ECDSA_BRP.der"; - pars.euicc_key_path := "./sgp26/eUICC/SK_EUICC_ECDSA_BRP.pem"; + var smdpp_ConnHdlrPars pars := f_init_pars(brainpool := true); var smdpp_ConnHdlr vc_conn; f_init(testcasename()); vc_conn := f_start_handler(refers(f_TC_GetBoundProfilePackage_01_Nominal), pars); @@ -5609,11 +5607,7 @@ } testcase TC_rsp_complete_flow_BRP() runs on MTC_CT { - var smdpp_ConnHdlrPars pars := f_init_pars(); - pars.smdp_es9p_server_port := mp_es9plus_server_port_brp; /* BRP server port */ - pars.cert_name_filter := "BRP"; /* Use Brainpool certificates */ - pars.euicc_cert_path := "./sgp26/eUICC/CERT_EUICC_ECDSA_BRP.der"; - pars.euicc_key_path := "./sgp26/eUICC/SK_EUICC_ECDSA_BRP.pem"; + var smdpp_ConnHdlrPars pars := f_init_pars(brainpool := true); var smdpp_ConnHdlr vc_conn; f_init(testcasename()); vc_conn := f_start_handler(refers(f_TC_rsp_complete_flow), pars); @@ -5622,11 +5616,7 @@ } testcase TC_SM_DP_ES9_HandleNotificationBRP() runs on MTC_CT { - var smdpp_ConnHdlrPars pars := f_init_pars(); - pars.smdp_es9p_server_port := mp_es9plus_server_port_brp; /* BRP server port */ - pars.cert_name_filter := "BRP"; /* Use Brainpool certificates */ - pars.euicc_cert_path := "./sgp26/eUICC/CERT_EUICC_ECDSA_BRP.der"; - pars.euicc_key_path := "./sgp26/eUICC/SK_EUICC_ECDSA_BRP.pem"; + var smdpp_ConnHdlrPars pars := f_init_pars(brainpool := true); var smdpp_ConnHdlr vc_conn; f_init(testcasename()); vc_conn := f_start_handler(refers(f_TC_HandleNotification_01_Nominal), pars); @@ -5635,11 +5625,7 @@ } testcase TC_SM_DP_ES9_CancelSession_After_AuthenticateClientBRP() runs on MTC_CT { - var smdpp_ConnHdlrPars pars := f_init_pars(); - pars.smdp_es9p_server_port := mp_es9plus_server_port_brp; /* BRP server port */ - pars.cert_name_filter := "BRP"; /* Use Brainpool certificates */ - pars.euicc_cert_path := "./sgp26/eUICC/CERT_EUICC_ECDSA_BRP.der"; - pars.euicc_key_path := "./sgp26/eUICC/SK_EUICC_ECDSA_BRP.pem"; + var smdpp_ConnHdlrPars pars := f_init_pars(brainpool := true); var smdpp_ConnHdlr vc_conn; f_init(testcasename()); vc_conn := f_start_handler(refers(f_TC_CancelSession_After_AuthenticateClient_01_End_User_Rejection), pars); @@ -5648,11 +5634,7 @@ } testcase TC_SM_DP_ES9_CancelSession_After_GetBoundProfilePackageBRP() runs on MTC_CT { - var smdpp_ConnHdlrPars pars := f_init_pars(); - pars.smdp_es9p_server_port := mp_es9plus_server_port_brp; /* BRP server port */ - pars.cert_name_filter := "BRP"; /* Use Brainpool certificates */ - pars.euicc_cert_path := "./sgp26/eUICC/CERT_EUICC_ECDSA_BRP.der"; - pars.euicc_key_path := "./sgp26/eUICC/SK_EUICC_ECDSA_BRP.pem"; + var smdpp_ConnHdlrPars pars := f_init_pars(brainpool := true); var smdpp_ConnHdlr vc_conn; f_init(testcasename()); vc_conn := f_start_handler(refers(f_TC_CancelSession_After_GetBoundProfilePackage_01), pars); -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41235?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: Ied4ffa6ff4b3efbbcfc9204ce8f21ea7ca4becf5 Gerrit-Change-Number: 41235 Gerrit-PatchSet: 1 Gerrit-Owner: dexter <pmaier(a)sysmocom.de>

1 0

[XS] Change in osmo-ttcn3-hacks[master]: smdpp_Tests: fix sourcecode formatting, remove line breaks
by dexter 21 Oct '25

21 Oct '25

dexter has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41232?usp=email ) Change subject: smdpp_Tests: fix sourcecode formatting, remove line breaks ...................................................................... smdpp_Tests: fix sourcecode formatting, remove line breaks Change-Id: I50175cd2989d6bfbe220c33e73a1724166ca548f Related: SYS#7339 --- M smdpp/smdpp_Tests.cfg M smdpp/smdpp_Tests.ttcn 2 files changed, 0 insertions(+), 3 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/32/41232/1 diff --git a/smdpp/smdpp_Tests.cfg b/smdpp/smdpp_Tests.cfg index 32bab0d..35f3140 100644 --- a/smdpp/smdpp_Tests.cfg +++ b/smdpp/smdpp_Tests.cfg @@ -15,8 +15,6 @@ #*.FileMask:=LOG_ALL | MATCHING | DEBUG; *.ConsoleMask:=ERROR | TESTCASE | USER | VERDICTOP | WARNING ; - - [TESTPORT_PARAMETERS] system.HTTP_server_port.KEYFILE := "../ipad/example_ca/pki/private/alttest.key" system.HTTP_server_port.CERTIFICATEFILE := "../ipad/example_ca/pki/issued/alttest.crt" diff --git a/smdpp/smdpp_Tests.ttcn b/smdpp/smdpp_Tests.ttcn index 79af08c..8ba125a 100644 --- a/smdpp/smdpp_Tests.ttcn +++ b/smdpp/smdpp_Tests.ttcn @@ -28,7 +28,6 @@ import from PKIX1Implicit88_Templates all; import from PKIX1Implicit88_Types all; - import from es12_Types_JSON all; import from es9p_Types_JSON all; import from es2p_Types_JSON all; -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41232?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: I50175cd2989d6bfbe220c33e73a1724166ca548f Gerrit-Change-Number: 41232 Gerrit-PatchSet: 1 Gerrit-Owner: dexter <pmaier(a)sysmocom.de>

1 0

[S] Change in osmo-ttcn3-hacks[master]: smdpp_Tests: make ES9+ and ES2+ ports configurable
by dexter 21 Oct '25

21 Oct '25

dexter has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41234?usp=email ) Change subject: smdpp_Tests: make ES9+ and ES2+ ports configurable ...................................................................... smdpp_Tests: make ES9+ and ES2+ ports configurable Change-Id: I17f2713088c583bec26a0a8cdff40c796e0a490b Related: SYS#7339 --- M smdpp/smdpp_Tests.cfg M smdpp/smdpp_Tests.ttcn 2 files changed, 15 insertions(+), 9 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/34/41234/1 diff --git a/smdpp/smdpp_Tests.cfg b/smdpp/smdpp_Tests.cfg index 35f3140..c5e9f49 100644 --- a/smdpp/smdpp_Tests.cfg +++ b/smdpp/smdpp_Tests.cfg @@ -20,6 +20,9 @@ system.HTTP_server_port.CERTIFICATEFILE := "../ipad/example_ca/pki/issued/alttest.crt" [MODULE_PARAMETERS] +smdpp_Tests.mp_es9plus_server_port_nist := 8000 +smdpp_Tests.mp_es9plus_server_port_brp := 8000 +smdpp_Tests.mp_es2plus_server_port := 8000 [MAIN_CONTROLLER] diff --git a/smdpp/smdpp_Tests.ttcn b/smdpp/smdpp_Tests.ttcn index 548b33a..579f742 100644 --- a/smdpp/smdpp_Tests.ttcn +++ b/smdpp/smdpp_Tests.ttcn @@ -385,8 +385,8 @@ * to match the FQDN in the test SSL certificates. To resolve the server FQDN to an IP address, an entry * in /etc/hosts is sufficient. */ smdp_server_fqdn := "testsmdpplus1.example.com", - smdp_es9p_server_port := 8000, - smdp_es2p_server_port := 8000, + smdp_es9p_server_port := mp_es9plus_server_port_nist, + smdp_es2p_server_port := mp_es2plus_server_port, cert_path := "./sgp26/CertificateIssuer", cert_name_filter := "", // Load all certificates (both NIST and BRP) or "NIST" or "BRP" euicc_cert_path := "./sgp26/eUICC/CERT_EUICC_ECDSA_NIST.der", @@ -408,6 +408,9 @@ modulepar { boolean mp_use_custom_tls_cert := true; charstring mp_custom_tls_cert_path := ""; // Empty means use cert pool only + integer mp_es9plus_server_port_nist := 8000; + integer mp_es9plus_server_port_brp := 8000; + integer mp_es2plus_server_port := 8000; } private altstep as_Tguard() runs on MTC_CT { @@ -5568,7 +5571,7 @@ testcase TC_SM_DP_ES9_InitiateAuthenticationBRP_01_Nominal() runs on MTC_CT { var smdpp_ConnHdlrPars pars := f_init_pars(); - pars.smdp_es9p_server_port := 8001; /* BRP server port */ + pars.smdp_es9p_server_port := mp_es9plus_server_port_brp; /* BRP server port */ pars.cert_name_filter := "BRP"; /* Use Brainpool certificates */ pars.euicc_cert_path := "./sgp26/eUICC/CERT_EUICC_ECDSA_BRP.der"; pars.euicc_key_path := "./sgp26/eUICC/SK_EUICC_ECDSA_BRP.pem"; @@ -5581,7 +5584,7 @@ testcase TC_SM_DP_ES9_AuthenticateClientBRP_01_Nominal() runs on MTC_CT { var smdpp_ConnHdlrPars pars := f_init_pars(); - pars.smdp_es9p_server_port := 8001; /* BRP server port */ + pars.smdp_es9p_server_port := mp_es9plus_server_port_brp; /* BRP server port */ pars.cert_name_filter := "BRP"; /* Use Brainpool certificates */ pars.euicc_cert_path := "./sgp26/eUICC/CERT_EUICC_ECDSA_BRP.der"; pars.euicc_key_path := "./sgp26/eUICC/SK_EUICC_ECDSA_BRP.pem"; @@ -5594,7 +5597,7 @@ testcase TC_SM_DP_ES9_GetBoundProfilePackageBRP_01_Nominal() runs on MTC_CT { var smdpp_ConnHdlrPars pars := f_init_pars(); - pars.smdp_es9p_server_port := 8001; /* BRP server port */ + pars.smdp_es9p_server_port := mp_es9plus_server_port_brp; /* BRP server port */ pars.cert_name_filter := "BRP"; /* Use Brainpool certificates */ pars.euicc_cert_path := "./sgp26/eUICC/CERT_EUICC_ECDSA_BRP.der"; pars.euicc_key_path := "./sgp26/eUICC/SK_EUICC_ECDSA_BRP.pem"; @@ -5607,7 +5610,7 @@ testcase TC_rsp_complete_flow_BRP() runs on MTC_CT { var smdpp_ConnHdlrPars pars := f_init_pars(); - pars.smdp_es9p_server_port := 8001; /* BRP server port */ + pars.smdp_es9p_server_port := mp_es9plus_server_port_brp; /* BRP server port */ pars.cert_name_filter := "BRP"; /* Use Brainpool certificates */ pars.euicc_cert_path := "./sgp26/eUICC/CERT_EUICC_ECDSA_BRP.der"; pars.euicc_key_path := "./sgp26/eUICC/SK_EUICC_ECDSA_BRP.pem"; @@ -5620,7 +5623,7 @@ testcase TC_SM_DP_ES9_HandleNotificationBRP() runs on MTC_CT { var smdpp_ConnHdlrPars pars := f_init_pars(); - pars.smdp_es9p_server_port := 8001; /* BRP server port */ + pars.smdp_es9p_server_port := mp_es9plus_server_port_brp; /* BRP server port */ pars.cert_name_filter := "BRP"; /* Use Brainpool certificates */ pars.euicc_cert_path := "./sgp26/eUICC/CERT_EUICC_ECDSA_BRP.der"; pars.euicc_key_path := "./sgp26/eUICC/SK_EUICC_ECDSA_BRP.pem"; @@ -5633,7 +5636,7 @@ testcase TC_SM_DP_ES9_CancelSession_After_AuthenticateClientBRP() runs on MTC_CT { var smdpp_ConnHdlrPars pars := f_init_pars(); - pars.smdp_es9p_server_port := 8001; /* BRP server port */ + pars.smdp_es9p_server_port := mp_es9plus_server_port_brp; /* BRP server port */ pars.cert_name_filter := "BRP"; /* Use Brainpool certificates */ pars.euicc_cert_path := "./sgp26/eUICC/CERT_EUICC_ECDSA_BRP.der"; pars.euicc_key_path := "./sgp26/eUICC/SK_EUICC_ECDSA_BRP.pem"; @@ -5646,7 +5649,7 @@ testcase TC_SM_DP_ES9_CancelSession_After_GetBoundProfilePackageBRP() runs on MTC_CT { var smdpp_ConnHdlrPars pars := f_init_pars(); - pars.smdp_es9p_server_port := 8001; /* BRP server port */ + pars.smdp_es9p_server_port := mp_es9plus_server_port_brp; /* BRP server port */ pars.cert_name_filter := "BRP"; /* Use Brainpool certificates */ pars.euicc_cert_path := "./sgp26/eUICC/CERT_EUICC_ECDSA_BRP.der"; pars.euicc_key_path := "./sgp26/eUICC/SK_EUICC_ECDSA_BRP.pem"; -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41234?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: I17f2713088c583bec26a0a8cdff40c796e0a490b Gerrit-Change-Number: 41234 Gerrit-PatchSet: 1 Gerrit-Owner: dexter <pmaier(a)sysmocom.de>

1 0

[S] Change in osmo-ttcn3-hacks[master]: smdpp_Tests: rename smdp_server_url to smdp_server_fqdn
by dexter 21 Oct '25

21 Oct '25

dexter has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41233?usp=email ) Change subject: smdpp_Tests: rename smdp_server_url to smdp_server_fqdn ...................................................................... smdpp_Tests: rename smdp_server_url to smdp_server_fqdn The member name "smdp_server_url" is technically not correct as the content is an FQDN. Let's rename it to "smdp_server_fqdn". Let's also put a comment in which we explain why FQDN is hardcoded and not available as configurable module parameter. Related: SYS#7339 Change-Id: I0224c838a368656821fff6014b0df7eb49466b0a --- M smdpp/smdpp_Tests.ttcn 1 file changed, 13 insertions(+), 10 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/33/41233/1 diff --git a/smdpp/smdpp_Tests.ttcn b/smdpp/smdpp_Tests.ttcn index 8ba125a..548b33a 100644 --- a/smdpp/smdpp_Tests.ttcn +++ b/smdpp/smdpp_Tests.ttcn @@ -360,7 +360,7 @@ } type record smdpp_ConnHdlrPars { - charstring smdp_server_url, + charstring smdp_server_fqdn, integer smdp_es9p_server_port, integer smdp_es2p_server_port, charstring cert_path, @@ -381,7 +381,10 @@ private function f_init_pars() runs on MTC_CT return smdpp_ConnHdlrPars { var smdpp_ConnHdlrPars pars := { - smdp_server_url := "testsmdpplus1.example.com", + /* Unfortunately the SMDP server FQDN cannot be made configurable via a module parameter since it always + * to match the FQDN in the test SSL certificates. To resolve the server FQDN to an IP address, an entry + * in /etc/hosts is sufficient. */ + smdp_server_fqdn := "testsmdpplus1.example.com", smdp_es9p_server_port := 8000, smdp_es2p_server_port := 8000, cert_path := "./sgp26/CertificateIssuer", @@ -444,7 +447,7 @@ ext_logInfo("Initializing RSP client"); g_rsp_client_handle := ext_RSPClient_create( - g_pars_smdpp.smdp_server_url, + g_pars_smdpp.smdp_server_fqdn, g_pars_smdpp.smdp_es9p_server_port, g_pars_smdpp.cert_path, g_pars_smdpp.cert_name_filter @@ -675,7 +678,7 @@ var RemoteProfileProvisioningRequest request := { initiateAuthenticationRequest := { euiccChallenge := g_euiccChallenge, - smdpAddress := g_pars_smdpp.smdp_server_url, + smdpAddress := g_pars_smdpp.smdp_server_fqdn, euiccInfo1 := euiccInfo1 } }; @@ -1204,7 +1207,7 @@ ) runs on smdpp_ConnHdlr return EuiccSigned1 { return { transactionId := g_transactionId, - serverAddress := g_pars_smdpp.smdp_server_url, + serverAddress := g_pars_smdpp.smdp_server_fqdn, serverChallenge := g_serverChallenge, euiccInfo2 := euiccInfo2, ctxParams1 := ctxParams @@ -1540,7 +1543,7 @@ var NotificationMetadata notifMeta := { seqNumber := 1, profileManagementOperation := c_notificationInstall, - notificationAddress := g_pars_smdpp.smdp_server_url, + notificationAddress := g_pars_smdpp.smdp_server_fqdn, iccid := omit }; @@ -1642,7 +1645,7 @@ var EuiccSigned1 euiccSigned1 := { transactionId := actualTransactionId, - serverAddress := g_pars_smdpp.smdp_server_url, + serverAddress := g_pars_smdpp.smdp_server_fqdn, serverChallenge := serverChallenge, euiccInfo2 := actualEuiccInfo2, ctxParams1 := ctxParams @@ -1993,7 +1996,7 @@ /* Build euiccSigned1 */ var EuiccSigned1 euiccSigned1 := { transactionId := transactionId, - serverAddress := g_pars_smdpp.smdp_server_url, + serverAddress := g_pars_smdpp.smdp_server_fqdn, serverChallenge := serverChallenge, euiccInfo2 := euiccInfo2, ctxParams1 := ctxParams @@ -3736,7 +3739,7 @@ var EuiccSigned1 euiccSig := { transactionId := g_transactionId, - serverAddress := g_pars_smdpp.smdp_server_url, + serverAddress := g_pars_smdpp.smdp_server_fqdn, serverChallenge := g_serverChallenge, euiccInfo2 := euic2, ctxParams1 := ctxpar @@ -4101,7 +4104,7 @@ var NotificationMetadata notifMeta := { seqNumber := 2, /* Different sequence number from PIR */ profileManagementOperation := c_notificationEnable, - notificationAddress := g_pars_smdpp.smdp_server_url, + notificationAddress := g_pars_smdpp.smdp_server_fqdn, iccid := omit /* No specific ICCID for this test */ }; -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41233?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: I0224c838a368656821fff6014b0df7eb49466b0a Gerrit-Change-Number: 41233 Gerrit-PatchSet: 1 Gerrit-Owner: dexter <pmaier(a)sysmocom.de>

1 0

[M] Change in osmo-ttcn3-hacks[master]: IPA: Introduce new OSMO extension TCAP_ROUTING
by lynxis lazus 21 Oct '25

21 Oct '25

Attention is currently required from: laforge, pespin. lynxis lazus has uploaded a new patch set (#4) to the change originally created by pespin. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41135?usp=email ) The following approvals got outdated and were removed: Code-Review+1 by laforge, Verified+1 by Jenkins Builder Change subject: IPA: Introduce new OSMO extension TCAP_ROUTING ...................................................................... IPA: Introduce new OSMO extension TCAP_ROUTING TCAP_ROUTING is an extension to IPA/SCCP where the node can requests to receive only certain TCAP traffic to support loadsharing. Related: SYS#5423 Change-Id: Ibfcf633f7bd8773f557901fdff7580f5879391a1 --- A library/IPA_EXT_TCAP_ROUTING.ttcn M library/IPA_Emulation.ttcnpp M library/IPA_Types.ttcn 3 files changed, 142 insertions(+), 1 deletion(-) git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/35/41135/4 -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41135?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newpatchset Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: Ibfcf633f7bd8773f557901fdff7580f5879391a1 Gerrit-Change-Number: 41135 Gerrit-PatchSet: 4 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Attention: laforge <laforge(a)osmocom.org> Gerrit-Attention: pespin <pespin(a)sysmocom.de>

1 0

[S] Change in libosmo-sigtran[master]: fix SCCP_MAX_OPTIONAL_DATA 128
by laforge 21 Oct '25

21 Oct '25

Attention is currently required from: neels, osmith. laforge has posted comments on this change by neels. ( https://gerrit.osmocom.org/c/libosmo-sigtran/+/41192?usp=email ) Change subject: fix SCCP_MAX_OPTIONAL_DATA 128 ...................................................................... Patch Set 1: Code-Review+2 (1 comment) Patchset: PS1: I would actually say this should als trigger a patch release of (at the very least) the mos recent libosmo-sigtran release. @osmith@sysmocom.de can you look into that, please? -- To view, visit https://gerrit.osmocom.org/c/libosmo-sigtran/+/41192?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: libosmo-sigtran Gerrit-Branch: master Gerrit-Change-Id: I841dbb2e8cbc13708ba7f5593f6bdd3b8da4329c Gerrit-Change-Number: 41192 Gerrit-PatchSet: 1 Gerrit-Owner: neels <nhofmeyr(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: osmith <osmith(a)sysmocom.de> Gerrit-Attention: neels <nhofmeyr(a)sysmocom.de> Gerrit-Comment-Date: Tue, 21 Oct 2025 11:09:31 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes

1 0

2025

2024

2023

2022

gerrit-log October 2025