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

[M] Change in osmo-mgw[master]: mgw: rtp-patch rfc5993hr: convert to each end's respective format
by falconia 22 Oct '25

22 Oct '25

Attention is currently required from: laforge, neels, pespin. falconia has posted comments on this change by falconia. ( https://gerrit.osmocom.org/c/osmo-mgw/+/39869?usp=email ) Change subject: mgw: rtp-patch rfc5993hr: convert to each end's respective format ...................................................................... Patch Set 2: (6 comments) Patchset: PS2: > > and also the BSC has to then be aware of the HR formats in use. […] I have so many objections to the argument raised by @nhofmeyr@sysmocom.de that I am not sure where to begin... Just a few thoughts that come to mind: * We all collectively, in our role as Osmocom community, got shot in the foot pretty badly when the decision was made, ages ago and long before I joined the project, to use MGCP as the control interface to OsmoMGW, as opposed to a completely custom, ad hoc and private interface. (For an example of the latter, see the control i/f to my tw-border-mgw.) Yes, I know the history behind this design decision: OsmoBSC was interfacing to ip.access proprietary MSC whose proprietary, non-3GPP flavor of A-over-IP interface (known around here as SCCPlite) includes MGCP with endpoints that function like rtpbridge in OsmoMGW. But this historical detail about the way some Osmocom-related commercial business once ran some years ago does nothing to help those of us who are 100% non-profit and FOSS-based (no proprietary MSCs kicking around, no commercial operations) - from my PoV, and from the standpoint of anyone else in a position similar to mine, the use of MGCP (and SDP) in Osmocom is an obstacle, a bone in the throat, an intrusion of IETF-isms into what is otherwise a clean world of GSM governed by 3GPP specs. * In terms of functionality, the best RTP format for HRv1 codec is TW-TS-002. For those who cannot or do not wish to use this extended format, the next best choice is RFC 5993 - and the latter is strictly, unambiguously stipulated by 3GPP for use at AoIP interface. The "raw" format of TS 101 318 is quite useful in internal interfaces (low-level library APIs), but use of this raw format in external RTP (external to any given single process) should be phased out. * The whole mechanism of on-the-fly HRv1 format conversion in OsmoMGW looks like a solution in search of a problem. What is wrong with simply configuring every OsmoBTS instance to use `rtp hr-format rfc5993` and thus match the requirement of AoIP interface? Is it about supporting ip.access nanoBTS that are stuck with TS 101 318 format? Or another pointless beauty contest, expending extra work just so that existing installations of osmo-bts-sysmo etc that default to `rtp hr-format ts101318` will convert to the right AoIP format "automagically"? * The idea keeps being tossed around that OsmoBSC should know which HR format each BTS uses and can therefore pass this info to OsmoMGW via an invented fmtp parameter. The part about inventing an fmtp parameter is trivial - I already did plenty of that for TW-TS-* - instead the problem is with BSC knowledge part. Let us all remember that the HR format emitted by OsmoBTS (all variants) is **configured by vty** on the BTS. How in the world would OsmoBSC know which format is configured by vty on each BTS? PS2: > I'd still want the unresolved comments to be adressed somehow. There exists a concept called "pick your battles", or "choose your battles wisely". Back in March I tried to do "the right thing" and prepare this patch for Osmocom, even though the feature in question is utterly useless - HRv1 codec has absolutely no use outside of just-for-fun retronetworking experiments, to the point where even American 2G Cooperative, a dedicated operator specifically for users of retro phones and probably the most retronetworking-oriented GSM operator anywhere in the world, won't be using HRv1 in production. And if someone does wish to play with HRv1 just for fun, there are much easier ways to accomplish that exercise, either by configuring `rtp hr-format rfc5993` in OsmoBTS or by enabling TW-TS-002 from MSC/CN side, which propagates down to the BTS and overrides the local `hr-format` setting. Therefore, I am driven to conclude that the present issue (OS#6059) and the whole debate around it is not about useful functionality, but more of a beauty contest, a question of which solution would be most aesthetically pleasing to @nhofmeyr@sysmocom.de, @pespin@sysmocom.de and other senior developers whose opinion ranks higher than mine. Therefore, unless those with opposing views soften their stance, I cannot justify spending more of my scarce volunteer time on this issue. File include/osmocom/mgcp/mgcp_rtp_end.h: https://gerrit.osmocom.org/c/osmo-mgw/+/39869/comment/435bc440_72a4b6c2?usp… : PS2, Line 40: bool hr_ts101318_detected; > I would prefer better layer separation for these: […] Multipart response, similar to the other comment where deep philosophical issues are involved: * Your proposal puts TW-TS-002 "on equals" with TS 101 318 and RFC 5993 in the enum, suggesting that you envision somehow converting between TW-TS-002 and those other formats. Such conversion is not possible and shows a fundamental misunderstanding: TW-TS-002 is not just a different representation format for the same information content, instead there is additional information content with TW-TS-002 that simply does not exist with either of the two "standard" formats. Hence there is no possibility of conversion. * The only correct way to handle TW-TS-002 is the way I handle it in my patch: treat it as a request from OsmoBSC that basically says "dear OsmoMGW, please stand down and disable all of your smart conversions, just pass the payload through and leave it alone". * The rest of your enum proposal I simply don't understand, and as already explained, I am running into a limit of how much volunteer work I can put into this issue that is purely for the heck of it, with no practical use that I can see. File src/libosmo-mgcp/mgcp_network.c: https://gerrit.osmocom.org/c/osmo-mgw/+/39869/comment/3a7150e2_dcc0c572?usp… : PS2, Line 715: static int rfc5993_hr_convert(struct msgb *msg, struct mgcp_conn_rtp *dst) > It is an artifact of how I developed this code in stages. […] Update 7 months later: I'll spin an updated patch with these arguments swapped. https://gerrit.osmocom.org/c/osmo-mgw/+/39869/comment/751287c5_c1ae1a13?usp… : PS2, Line 1577: Th > s/This internal function examines/Examine/ What language do you then propose for the "it is invoked when..." part? https://gerrit.osmocom.org/c/osmo-mgw/+/39869/comment/255bde41_ba255622?usp… : PS2, Line 1658: Autodetect > s/between // ? Removing "between" would make the imperative sentence ungrammatical. -- To view, visit https://gerrit.osmocom.org/c/osmo-mgw/+/39869?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-mgw Gerrit-Branch: master Gerrit-Change-Id: I6b446ad83c540fb8b7e1aae24b78c27010212d64 Gerrit-Change-Number: 39869 Gerrit-PatchSet: 2 Gerrit-Owner: falconia <falcon(a)freecalypso.org> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: neels <nhofmeyr(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: neels <nhofmeyr(a)sysmocom.de> Gerrit-Attention: laforge <laforge(a)osmocom.org> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Wed, 22 Oct 2025 04:00:46 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: falconia <falcon(a)freecalypso.org> Comment-In-Reply-To: neels <nhofmeyr(a)sysmocom.de> Comment-In-Reply-To: laforge <laforge(a)osmocom.org> Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>

1 0

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

22 Oct '25

Attention is currently required from: lynxis lazus. pespin has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41135?usp=email ) Change subject: IPA: Introduce new OSMO extension TCAP_ROUTING ...................................................................... Patch Set 4: Code-Review+1 -- 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: comment 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-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Comment-Date: Tue, 21 Oct 2025 22:04:59 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[XS] Change in ...pyosmocom[master]: utils: h2i(): use list() to convert bytes to integers
by laforge 21 Oct '25

21 Oct '25

laforge has submitted this change. ( https://gerrit.osmocom.org/c/python/pyosmocom/+/39454?usp=email ) ( 1 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one. )Change subject: utils: h2i(): use list() to convert bytes to integers ...................................................................... utils: h2i(): use list() to convert bytes to integers Change-Id: Icb0d0803b7ae4e0b3a292ba96f58c26d0ca88abd --- M src/osmocom/utils.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: lynxis lazus: Looks good to me, approved Jenkins Builder: Verified laforge: Looks good to me, but someone else must approve diff --git a/src/osmocom/utils.py b/src/osmocom/utils.py index 6c823ec..347bb93 100644 --- a/src/osmocom/utils.py +++ b/src/osmocom/utils.py @@ -78,7 +78,7 @@ def h2i(s: Hexstr) -> List[int]: """convert from a string of hex nibbles to a list of integers""" - return [(int(x, 16) << 4)+int(y, 16) for x, y in zip(s[0::2], s[1::2])] + return list(h2b(s)) def i2h(s: List[int]) -> hexstr: -- To view, visit https://gerrit.osmocom.org/c/python/pyosmocom/+/39454?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: python/pyosmocom Gerrit-Branch: master Gerrit-Change-Id: Icb0d0803b7ae4e0b3a292ba96f58c26d0ca88abd Gerrit-Change-Number: 39454 Gerrit-PatchSet: 2 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: lynxis lazus <lynxis(a)fe80.eu>

1 0

[L] Change in osmo-ttcn3-hacks[master]: hlr: pyhss: create/delete subscribers
by laforge 21 Oct '25

21 Oct '25

Attention is currently required from: daniel, dexter, fixeria, osmith. laforge has posted comments on this change by osmith. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41227?usp=email ) Change subject: hlr: pyhss: create/delete subscribers ...................................................................... Patch Set 2: Code-Review+2 (1 comment) File library/PyHSS_REST_Functions.ttcn: https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41227/comment/d68fd0ce_f22a… : PS2, Line 43: private function f_pyhss_algo(template (omit) charstring algo2g := omit, > > In that case, the variable in line 84 would always be set to the algo2g value? […] Done -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41227?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: Ic924dabbc813459f73d6646ee17b79cb11d39a76 Gerrit-Change-Number: 41227 Gerrit-PatchSet: 2 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-CC: daniel <dwillmann(a)sysmocom.de> Gerrit-Attention: osmith <osmith(a)sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Attention: daniel <dwillmann(a)sysmocom.de> Gerrit-Attention: dexter <pmaier(a)sysmocom.de> Gerrit-Comment-Date: Tue, 21 Oct 2025 20:30:05 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Comment-In-Reply-To: osmith <osmith(a)sysmocom.de> Comment-In-Reply-To: daniel <dwillmann(a)sysmocom.de>

1 0

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

21 Oct '25

Attention is currently required from: lynxis lazus, pespin. laforge has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41135?usp=email ) Change subject: IPA: Introduce new OSMO extension TCAP_ROUTING ...................................................................... Patch Set 4: Code-Review+1 -- 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: comment 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: pespin <pespin(a)sysmocom.de> Gerrit-Attention: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Comment-Date: Tue, 21 Oct 2025 20:28:45 +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 laforge 21 Oct '25

21 Oct '25

laforge has submitted this change. ( 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(-) Approvals: Jenkins Builder: Verified osmith: Looks good to me, approved diff --git a/smdpp/smdpp_Tests.ttcn b/smdpp/smdpp_Tests.ttcn index d40d6df..7a300a2 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: merged Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: Ied4ffa6ff4b3efbbcfc9204ce8f21ea7ca4becf5 Gerrit-Change-Number: 41235 Gerrit-PatchSet: 2 Gerrit-Owner: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>

1 0

[XS] Change in ...pyosmocom[master]: utils: h2i(): use list() to convert bytes to integers
by laforge 21 Oct '25

21 Oct '25

Attention is currently required from: fixeria. laforge has posted comments on this change by fixeria. ( https://gerrit.osmocom.org/c/python/pyosmocom/+/39454?usp=email ) Change subject: utils: h2i(): use list() to convert bytes to integers ...................................................................... Patch Set 2: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/python/pyosmocom/+/39454?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: python/pyosmocom Gerrit-Branch: master Gerrit-Change-Id: Icb0d0803b7ae4e0b3a292ba96f58c26d0ca88abd Gerrit-Change-Number: 39454 Gerrit-PatchSet: 2 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Comment-Date: Tue, 21 Oct 2025 20:27:35 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[XS] Change in ...pyosmocom[master]: utils: b2h()/i2h(): use bytes.hex() API
by laforge 21 Oct '25

21 Oct '25

laforge has submitted this change. ( https://gerrit.osmocom.org/c/python/pyosmocom/+/39453?usp=email ) Change subject: utils: b2h()/i2h(): use bytes.hex() API ...................................................................... utils: b2h()/i2h(): use bytes.hex() API Change-Id: Iaef25f614e7a4b1c3eb328e560bf9a300f70ae31 --- M src/osmocom/utils.py 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Jenkins Builder: Verified lynxis lazus: Looks good to me, approved dexter: Looks good to me, approved diff --git a/src/osmocom/utils.py b/src/osmocom/utils.py index 5947453..6c823ec 100644 --- a/src/osmocom/utils.py +++ b/src/osmocom/utils.py @@ -73,7 +73,7 @@ def b2h(b: bytearray) -> hexstr: """convert from a sequence of bytes to a string of hex nibbles""" - return hexstr(''.join(['%02x' % (x) for x in b])) + return hexstr(b.hex()) def h2i(s: Hexstr) -> List[int]: @@ -83,7 +83,7 @@ def i2h(s: List[int]) -> hexstr: """convert from a list of integers to a string of hex nibbles""" - return hexstr(''.join(['%02x' % (x) for x in s])) + return hexstr(bytes(s).hex()) def h2s(s: Hexstr) -> str: -- To view, visit https://gerrit.osmocom.org/c/python/pyosmocom/+/39453?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: python/pyosmocom Gerrit-Branch: master Gerrit-Change-Id: Iaef25f614e7a4b1c3eb328e560bf9a300f70ae31 Gerrit-Change-Number: 39453 Gerrit-PatchSet: 2 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: lynxis lazus <lynxis(a)fe80.eu>

1 0

[M] Change in osmo-mgw[master]: mgw: rtp-patch rfc5993hr: convert to each end's respective format
by laforge 21 Oct '25

21 Oct '25

Attention is currently required from: falconia, neels, pespin. laforge has posted comments on this change by falconia. ( https://gerrit.osmocom.org/c/osmo-mgw/+/39869?usp=email ) Change subject: mgw: rtp-patch rfc5993hr: convert to each end's respective format ...................................................................... Patch Set 2: -Code-Review (1 comment) Patchset: PS2: I'd still want the unresolved comments to be adressed somehow. -- To view, visit https://gerrit.osmocom.org/c/osmo-mgw/+/39869?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-mgw Gerrit-Branch: master Gerrit-Change-Id: I6b446ad83c540fb8b7e1aae24b78c27010212d64 Gerrit-Change-Number: 39869 Gerrit-PatchSet: 2 Gerrit-Owner: falconia <falcon(a)freecalypso.org> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: neels <nhofmeyr(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: falconia <falcon(a)freecalypso.org> Gerrit-Attention: neels <nhofmeyr(a)sysmocom.de> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Tue, 21 Oct 2025 20:25:37 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes

1 0

[M] Change in osmo-mgw[master]: mgw: rtp-patch rfc5993hr: convert to each end's respective format
by laforge 21 Oct '25

21 Oct '25

Attention is currently required from: falconia, neels, pespin. laforge has posted comments on this change by falconia. ( https://gerrit.osmocom.org/c/osmo-mgw/+/39869?usp=email ) Change subject: mgw: rtp-patch rfc5993hr: convert to each end's respective format ...................................................................... Patch Set 2: Code-Review+1 (4 comments) File src/libosmo-mgcp/mgcp_network.c: https://gerrit.osmocom.org/c/osmo-mgw/+/39869/comment/7a37b698_3f7988da?usp… : PS2, Line 691: * consider the full RTP chain of a single call: > The comment by @laforge@osmocom. […] Done https://gerrit.osmocom.org/c/osmo-mgw/+/39869/comment/1e1de471_7286ebf5?usp… : PS2, Line 698: will always be RFC 5993 > > On AoIP, there is no discussion. […] Done https://gerrit.osmocom.org/c/osmo-mgw/+/39869/comment/a0cf7996_312b9e4a?usp… : PS2, Line 1660: strcmp(conn_src->end.cset.codec->subtype_name, "GSM-HR-08") == 0 && conn_src->end.rfc5993_hr_convert) { > I originally had it split, but the overzealous linter forced me to join into this longer line! See t […] Done https://gerrit.osmocom.org/c/osmo-mgw/+/39869/comment/dfea6621_0f340ae1?usp… : PS2, Line 1661: int rc = examine_rtp_rx_gsmhr(msg, conn_src); > It is called for every received packet. […] Done -- To view, visit https://gerrit.osmocom.org/c/osmo-mgw/+/39869?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-mgw Gerrit-Branch: master Gerrit-Change-Id: I6b446ad83c540fb8b7e1aae24b78c27010212d64 Gerrit-Change-Number: 39869 Gerrit-PatchSet: 2 Gerrit-Owner: falconia <falcon(a)freecalypso.org> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: neels <nhofmeyr(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: falconia <falcon(a)freecalypso.org> Gerrit-Attention: neels <nhofmeyr(a)sysmocom.de> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Tue, 21 Oct 2025 20:25:11 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Comment-In-Reply-To: falconia <falcon(a)freecalypso.org> Comment-In-Reply-To: neels <nhofmeyr(a)sysmocom.de> Comment-In-Reply-To: laforge <laforge(a)osmocom.org> Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>

1 0