- gerrit-log - lists.osmocom.org

[S] Change in pysim[master]: Print SMSC in pySim-read.py
by laforge 31 Jan '26

31 Jan '26

laforge has submitted this change. ( https://gerrit.osmocom.org/c/pysim/+/41786?usp=email ) Change subject: Print SMSC in pySim-read.py ...................................................................... Print SMSC in pySim-read.py Change-Id: I17067b68086316d51fd71ba77049874605594e3f --- M pySim-read.py M tests/pySim-prog_test/Fairwaves-SIM.ok M tests/pySim-prog_test/Wavemobile-SIM.ok M tests/pySim-prog_test/fakemagicsim.ok M tests/pySim-prog_test/sysmoISIM-SJA2.ok M tests/pySim-prog_test/sysmoISIM-SJA5.ok M tests/pySim-prog_test/sysmoUSIM-SJS1.ok M tests/pySim-prog_test/sysmosim-gr1.ok 8 files changed, 17 insertions(+), 0 deletions(-) Approvals: laforge: Looks good to me, approved Jenkins Builder: Verified dexter: Looks good to me, but someone else must approve; Verified diff --git a/pySim-read.py b/pySim-read.py index 9829a15..481e9da 100755 --- a/pySim-read.py +++ b/pySim-read.py @@ -44,6 +44,7 @@ from pySim.legacy.cards import card_detect, SimCard, UsimCard, IsimCard from pySim.utils import dec_imsi, dec_iccid from pySim.legacy.utils import format_xplmn_w_act, dec_st, dec_msisdn +from pySim.ts_51_011 import EF_SMSP option_parser = argparse.ArgumentParser(description='Legacy tool for reading some parts of a SIM card', formatter_class=argparse.ArgumentDefaultsHelpFormatter) @@ -141,6 +142,15 @@ (res, sw) = card.read_record('SMSP', 1) if sw == '9000': print("SMSP: %s" % (res,)) + ef_smsp = EF_SMSP() + smsc_a = ef_smsp.decode_record_bin(h2b(res), 1).get('tp_sc_addr', {}) + smsc_n = smsc_a.get('call_number', None) + if smsc_a.get('ton_npi', {}).get('type_of_number', None) == 'international' and smsc_n is not None: + smsc = '+' + smsc_n + else: + smsc = smsc_n + if smsc is not None: + print("SMSC: %s" % (smsc,)) else: print("SMSP: Can't read, response code = %s" % (sw,)) diff --git a/tests/pySim-prog_test/Fairwaves-SIM.ok b/tests/pySim-prog_test/Fairwaves-SIM.ok index 3751c57..303e8ae 100644 --- a/tests/pySim-prog_test/Fairwaves-SIM.ok +++ b/tests/pySim-prog_test/Fairwaves-SIM.ok @@ -6,6 +6,7 @@ GID1: ffffffffffffffff GID2: ffffffffffffffff SMSP: e1ffffffffffffffffffffffff0581005155f5ffffffffffff000000ffffffffffffffffffffffffffff +SMSC: 0015555 SPN: Fairwaves Show in HPLMN: False Hide in OPLMN: False diff --git a/tests/pySim-prog_test/Wavemobile-SIM.ok b/tests/pySim-prog_test/Wavemobile-SIM.ok index 7a0ecec..3c1e62d 100644 --- a/tests/pySim-prog_test/Wavemobile-SIM.ok +++ b/tests/pySim-prog_test/Wavemobile-SIM.ok @@ -6,6 +6,7 @@ GID1: Can't read file -- SW match failed! Expected 9000 and got 6a82. GID2: Can't read file -- SW match failed! Expected 9000 and got 6a82. SMSP: e1ffffffffffffffffffffffff0581005155f5ffffffffffff000000ffffffffffffffffffffffffffff +SMSC: 0015555 SPN: wavemobile Show in HPLMN: False Hide in OPLMN: False diff --git a/tests/pySim-prog_test/fakemagicsim.ok b/tests/pySim-prog_test/fakemagicsim.ok index 4c366f9..306cef9 100644 --- a/tests/pySim-prog_test/fakemagicsim.ok +++ b/tests/pySim-prog_test/fakemagicsim.ok @@ -6,6 +6,7 @@ GID1: Can't read file -- SW match failed! Expected 9000 and got 9404. GID2: Can't read file -- SW match failed! Expected 9000 and got 9404. SMSP: ffffffffffffffffffffffffe1ffffffffffffffffffffffff0581005155f5ffffffffffff000000 +SMSC: 0015555 SPN: Magic Show in HPLMN: True Hide in OPLMN: False diff --git a/tests/pySim-prog_test/sysmoISIM-SJA2.ok b/tests/pySim-prog_test/sysmoISIM-SJA2.ok index 5640171..5e2b45f 100644 --- a/tests/pySim-prog_test/sysmoISIM-SJA2.ok +++ b/tests/pySim-prog_test/sysmoISIM-SJA2.ok @@ -6,6 +6,7 @@ GID1: ffffffffffffffffffff GID2: ffffffffffffffffffff SMSP: ffffffffffffffffffffffffffffffffffffffffffffffffe1ffffffffffffffffffffffff0581005155f5ffffffffffff000000 +SMSC: 0015555 SPN: Magic Show in HPLMN: True Hide in OPLMN: True diff --git a/tests/pySim-prog_test/sysmoISIM-SJA5.ok b/tests/pySim-prog_test/sysmoISIM-SJA5.ok index 28b2875..b6d7862 100644 --- a/tests/pySim-prog_test/sysmoISIM-SJA5.ok +++ b/tests/pySim-prog_test/sysmoISIM-SJA5.ok @@ -6,6 +6,7 @@ GID1: ffffffffffffffffffff GID2: ffffffffffffffffffff SMSP: ffffffffffffffffffffffffffffffffffffffffffffffffe1ffffffffffffffffffffffff0581005155f5ffffffffffff000000 +SMSC: 0015555 SPN: Magic Show in HPLMN: True Hide in OPLMN: True diff --git a/tests/pySim-prog_test/sysmoUSIM-SJS1.ok b/tests/pySim-prog_test/sysmoUSIM-SJS1.ok index 2f48c2c..c669353 100644 --- a/tests/pySim-prog_test/sysmoUSIM-SJS1.ok +++ b/tests/pySim-prog_test/sysmoUSIM-SJS1.ok @@ -6,6 +6,7 @@ GID1: ffffffffffffffffffff GID2: ffffffffffffffffffff SMSP: ffffffffffffffffffffffffffffffffffffffffffffffffe1ffffffffffffffffffffffff0581005155f5ffffffffffff000000 +SMSC: 0015555 SPN: Magic Show in HPLMN: True Hide in OPLMN: True diff --git a/tests/pySim-prog_test/sysmosim-gr1.ok b/tests/pySim-prog_test/sysmosim-gr1.ok index 03b6a17..3d926e2 100644 --- a/tests/pySim-prog_test/sysmosim-gr1.ok +++ b/tests/pySim-prog_test/sysmosim-gr1.ok @@ -6,6 +6,7 @@ GID1: Can't read file -- SW match failed! Expected 9000 and got 9404. GID2: Can't read file -- SW match failed! Expected 9000 and got 9404. SMSP: ffffffffffffffffffffffffe1ffffffffffffffffffffffff0581005155f5ffffffffffff000000 +SMSC: 0015555 SPN: Not available Show in HPLMN: False Hide in OPLMN: False -- To view, visit https://gerrit.osmocom.org/c/pysim/+/41786?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: I17067b68086316d51fd71ba77049874605594e3f Gerrit-Change-Number: 41786 Gerrit-PatchSet: 16 Gerrit-Owner: phcoder <phcoder(a)gmail.com> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de>

1 0

[S] Change in pysim[master]: Print SMSC in pySim-read.py
by laforge 31 Jan '26

31 Jan '26

Attention is currently required from: fixeria, phcoder. laforge has posted comments on this change by phcoder. ( https://gerrit.osmocom.org/c/pysim/+/41786?usp=email ) Change subject: Print SMSC in pySim-read.py ...................................................................... Patch Set 15: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/pysim/+/41786?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: I17067b68086316d51fd71ba77049874605594e3f Gerrit-Change-Number: 41786 Gerrit-PatchSet: 15 Gerrit-Owner: phcoder <phcoder(a)gmail.com> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Attention: phcoder <phcoder(a)gmail.com> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Comment-Date: Sat, 31 Jan 2026 01:30:30 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[XS] Change in pysim[master]: es9p_client: Fix type conversion in installation result notification
by laforge 31 Jan '26

31 Jan '26

laforge has posted comments on this change by laforge. ( https://gerrit.osmocom.org/c/pysim/+/41973?usp=email ) Change subject: es9p_client: Fix type conversion in installation result notification ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/pysim/+/41973?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: I83ad047e043dc6b3462b188ce6dd0b2cc0e52e87 Gerrit-Change-Number: 41973 Gerrit-PatchSet: 1 Gerrit-Owner: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Comment-Date: Sat, 31 Jan 2026 01:23:09 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[S] Change in pysim[master]: pySim-trace: pySim.apdu_source.stdin_hex
by laforge 31 Jan '26

31 Jan '26

laforge has submitted this change. ( https://gerrit.osmocom.org/c/pysim/+/37925?usp=email ) Change subject: pySim-trace: pySim.apdu_source.stdin_hex ...................................................................... pySim-trace: pySim.apdu_source.stdin_hex This introduces an "APDU source" for pySim-trace which enables the decoding of APDUs that are copy+pasted from elsewhere, for example APDU logs in text form created by proprietary tools, or to decode personalization scripts or the like. Change-Id: I5aacf13b7c27cea9efd42f01dacca61068c3aa33 --- M pySim-trace.py A pySim/apdu_source/stdin_hex.py 2 files changed, 46 insertions(+), 0 deletions(-) Approvals: dexter: Looks good to me, approved; Verified laforge: Looks good to me, approved Jenkins Builder: Verified diff --git a/pySim-trace.py b/pySim-trace.py index 09e9584..0aa1e9c 100755 --- a/pySim-trace.py +++ b/pySim-trace.py @@ -23,6 +23,7 @@ from pySim.apdu_source.pyshark_rspro import PysharkRsproPcap, PysharkRsproLive from pySim.apdu_source.pyshark_gsmtap import PysharkGsmtapPcap from pySim.apdu_source.tca_loader_log import TcaLoaderLogApduSource +from pySim.apdu_source.stdin_hex import StdinHexApduSource from pySim.apdu.ts_102_221 import UiccSelect, UiccStatus @@ -190,6 +191,10 @@ parser_tcaloader_log.add_argument('-f', '--log-file', required=True, help='Name of the log file to be read') +parser_stdin_hex = subparsers.add_parser('stdin-hex', help=""" + Read APDUs as hex-string from stdin.""") + + if __name__ == '__main__': opts = option_parser.parse_args() @@ -205,6 +210,8 @@ s = PysharkGsmtapPcap(opts.pcap_file) elif opts.source == 'tca-loader-log': s = TcaLoaderLogApduSource(opts.log_file) + elif opts.source == 'stdin-hex': + s = StdinHexApduSource() else: raise ValueError("unsupported source %s", opts.source) diff --git a/pySim/apdu_source/stdin_hex.py b/pySim/apdu_source/stdin_hex.py new file mode 100644 index 0000000..67eeb7d --- /dev/null +++ b/pySim/apdu_source/stdin_hex.py @@ -0,0 +1,39 @@ +# coding=utf-8 + +# (C) 2024 by Harald Welte <laforge(a)osmocom.org> +# +# This program is free software: you can redistribute it and/or modify +# it under the terms of the GNU General Public License as published by +# the Free Software Foundation, either version 2 of the License, or +# (at your option) any later version. +# +# This program is distributed in the hope that it will be useful, +# but WITHOUT ANY WARRANTY; without even the implied warranty of +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +# GNU General Public License for more details. +# +# You should have received a copy of the GNU General Public License +# along with this program. If not, see <http://www.gnu.org/licenses/>. + + +from pySim.utils import h2b + +from pySim.apdu.ts_102_221 import ApduCommands as UiccApduCommands +from pySim.apdu.ts_102_222 import ApduCommands as UiccAdmApduCommands +from pySim.apdu.ts_31_102 import ApduCommands as UsimApduCommands +from pySim.apdu.global_platform import ApduCommands as GpApduCommands + +from . import ApduSource, PacketType, CardReset + +ApduCommands = UiccApduCommands + UiccAdmApduCommands + UsimApduCommands + GpApduCommands + +class StdinHexApduSource(ApduSource): + """ApduSource for reading apdu hex-strings from stdin.""" + + def read_packet(self) -> PacketType: + while True: + command = input("C-APDU >") + if len(command) == 0: + continue + response = '9000' + return ApduCommands.parse_cmd_bytes(h2b(command) + h2b(response)) -- To view, visit https://gerrit.osmocom.org/c/pysim/+/37925?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: I5aacf13b7c27cea9efd42f01dacca61068c3aa33 Gerrit-Change-Number: 37925 Gerrit-PatchSet: 8 Gerrit-Owner: laforge <laforge(a)osmocom.org> 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>

1 0

[XS] Change in docker-playground[master]: osmo-ns-master: drop python3-osmopy-utils / --enable-external-tests
by pespin 30 Jan '26

30 Jan '26

Attention is currently required from: fixeria, osmith. pespin has posted comments on this change by fixeria. ( https://gerrit.osmocom.org/c/docker-playground/+/41996?usp=email ) Change subject: osmo-ns-master: drop python3-osmopy-utils / --enable-external-tests ...................................................................... Patch Set 1: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/docker-playground/+/41996?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: docker-playground Gerrit-Branch: master Gerrit-Change-Id: I342099be87175dfe189a8906b53ee26328c0a417 Gerrit-Change-Number: 41996 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: osmith <osmith(a)sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Comment-Date: Fri, 30 Jan 2026 21:30:30 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[XS] Change in docker-playground[master]: osmo-ns-master: drop python3-osmopy-utils / --enable-external-tests
by fixeria 30 Jan '26

30 Jan '26

fixeria has uploaded this change for review. ( https://gerrit.osmocom.org/c/docker-playground/+/41996?usp=email ) Change subject: osmo-ns-master: drop python3-osmopy-utils / --enable-external-tests ...................................................................... osmo-ns-master: drop python3-osmopy-utils / --enable-external-tests --enable-external-tests requires python3-osmopy-utils, however this package is no longer available for debian:10. We're not running tests, so it's not really needed. Removing it fixes ttcn3-ns-test-fr. Change-Id: I342099be87175dfe189a8906b53ee26328c0a417 --- M osmo-ns-master/Dockerfile 1 file changed, 1 insertion(+), 2 deletions(-) git pull ssh://gerrit.osmocom.org:29418/docker-playground refs/changes/96/41996/1 diff --git a/osmo-ns-master/Dockerfile b/osmo-ns-master/Dockerfile index 35e360d..a76e459 100644 --- a/osmo-ns-master/Dockerfile +++ b/osmo-ns-master/Dockerfile @@ -8,7 +8,6 @@ debian*) \ apt-get update && \ apt-get install -y --no-install-recommends \ - python3-osmopy-utils \ libmnl-dev && \ apt-get clean \ ;; \ @@ -31,7 +30,7 @@ (git symbolic-ref -q HEAD && git reset --hard origin/$LIBOSMOCORE_BRANCH || exit 1); \ git rev-parse --abbrev-ref HEAD && git rev-parse HEAD && \ autoreconf -fi && \ - CONFIGURE_FLAGS="--disable-doxygen --disable-pcsc --enable-external-tests" && \ + CONFIGURE_FLAGS="--disable-doxygen --disable-pcsc" && \ if [ "$DISTRO" = "debian-buster" ]; then \ CONFIGURE_FLAGS="$CONFIGURE_FLAGS --disable-uring"; \ fi && \ -- To view, visit https://gerrit.osmocom.org/c/docker-playground/+/41996?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: docker-playground Gerrit-Branch: master Gerrit-Change-Id: I342099be87175dfe189a8906b53ee26328c0a417 Gerrit-Change-Number: 41996 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>

1 0

[XS] Change in docker-playground[master]: ttcn3-bts-test: fail on unexpected respawn count
by pespin 30 Jan '26

30 Jan '26

pespin has submitted this change. ( https://gerrit.osmocom.org/c/docker-playground/+/41981?usp=email ) Change subject: ttcn3-bts-test: fail on unexpected respawn count ...................................................................... ttcn3-bts-test: fail on unexpected respawn count Suggested-by: Pau Espin Pedrol <pespin(a)sysmocom.de> Change-Id: Idd9f8dcf1a7186a0715c1026da44f404c1c574d9 --- M ttcn3-bts-test/jenkins.sh 1 file changed, 3 insertions(+), 2 deletions(-) Approvals: Jenkins Builder: Verified pespin: Looks good to me, but someone else must approve fixeria: Looks good to me, approved diff --git a/ttcn3-bts-test/jenkins.sh b/ttcn3-bts-test/jenkins.sh index 6d738c4..6f75266 100755 --- a/ttcn3-bts-test/jenkins.sh +++ b/ttcn3-bts-test/jenkins.sh @@ -279,7 +279,7 @@ docker_kill_wait ${BUILD_TAG}-bts } -show_respawn_count() { +check_respawn_count() { set +x local count=$(grep -P 'respawn: \d+: starting: ' "$VOL_BASE_DIR"/bts/osmo-bts.log | wc -l) # Currently we run generic/bts/oml configurations, which means 3 @@ -298,6 +298,7 @@ echo "================================================================" echo printf '\033[0m' + exit 1 } network_create @@ -346,4 +347,4 @@ # Show respawn count at the very end clean_up_common -show_respawn_count +check_respawn_count -- To view, visit https://gerrit.osmocom.org/c/docker-playground/+/41981?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: docker-playground Gerrit-Branch: master Gerrit-Change-Id: Idd9f8dcf1a7186a0715c1026da44f404c1c574d9 Gerrit-Change-Number: 41981 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>

1 0

[L] Change in pysim[master]: personalization: add param_source.py, implement batch personalization
by laforge 30 Jan '26

30 Jan '26

Attention is currently required from: neels. laforge has posted comments on this change by neels. ( https://gerrit.osmocom.org/c/pysim/+/40096?usp=email ) Change subject: personalization: add param_source.py, implement batch personalization ...................................................................... Patch Set 10: (3 comments) File pySim/esim/saip/param_source.py: https://gerrit.osmocom.org/c/pysim/+/40096/comment/b72f1045_0a1b2da0?usp=em… : PS10, Line 50: This is dead code. You would just write `pass` here as it is impossible to instantiate an instance of an ABC-inheriting class without overriding all the abstractmethods in that derived class. https://gerrit.osmocom.org/c/pysim/+/40096/comment/e47f088e_2679bd4c?usp=em… : PS10, Line 84: likewise here, just `pass` will do. File pySim/esim/saip/personalization.py: https://gerrit.osmocom.org/c/pysim/+/40096/comment/9c3482f5_4e1f1342?usp=em… : PS10, Line 705: def __init__(self, highly unusual formatting which we don't do this way in any other part o the code. Do we really need those kind of cosmetic experiments? Also, we use spaces between name of the parameter and the type annotation. I'm not sure why code added doesn't just by default always uses the surrounding style without any attempt to do something different. -- To view, visit https://gerrit.osmocom.org/c/pysim/+/40096?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: I497c60c101ea0eea980e8b1a4b1f36c0eda39002 Gerrit-Change-Number: 40096 Gerrit-PatchSet: 10 Gerrit-Owner: neels <nhofmeyr(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-CC: laforge <laforge(a)osmocom.org> Gerrit-Attention: neels <nhofmeyr(a)sysmocom.de> Gerrit-Comment-Date: Fri, 30 Jan 2026 19:38:26 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No

1 0

[S] Change in pysim[master]: personalization: set some typical parameter names
by laforge 30 Jan '26

30 Jan '26

laforge has submitted this change. ( https://gerrit.osmocom.org/c/pysim/+/40093?usp=email ) Change subject: personalization: set some typical parameter names ...................................................................... personalization: set some typical parameter names These names better match what humans expect to read, for example "PIN1" instead of "Pin1". (We still fall back to the __class__.__name__ if a subclass omits a specific name, see the ConfigurableParameter init.) Change-Id: I31f390d634e58c384589c50a33ca45d6f86d4e10 --- M pySim/esim/saip/personalization.py 1 file changed, 16 insertions(+), 1 deletion(-) Approvals: laforge: Looks good to me, approved Jenkins Builder: Verified diff --git a/pySim/esim/saip/personalization.py b/pySim/esim/saip/personalization.py index 75a43aa..943784f 100644 --- a/pySim/esim/saip/personalization.py +++ b/pySim/esim/saip/personalization.py @@ -303,6 +303,7 @@ class Iccid(DecimalParam): """ICCID Parameter. Input: string of decimal digits. If the string of digits is only 18 digits long, add a Luhn check digit.""" + name = 'ICCID' min_len = 18 max_len = 20 @@ -321,6 +322,8 @@ class Imsi(DecimalParam): """Configurable IMSI. Expects value to be a string of digits. Automatically sets the ACC to the last digit of the IMSI.""" + + name = 'IMSI' min_len = 6 max_len = 15 @@ -339,12 +342,13 @@ """Configurable SMSC (SMS Service Centre) TP-SC-ADDR. Expects to be a phone number in national or international format (designated by a leading +). Automatically sets the NPI to E.164 and the TON based on presence or absence of leading +.""" + + name = 'SMSP-TP-SC-ADDR' allow_chars = '+0123456789' strip_chars = ' \t\r\n' max_len = 21 # '+' and 20 digits min_len = 1 - @classmethod def validate_val(cls, val): val = super().validate_val(val) @@ -507,9 +511,11 @@ f" cannot find pukCode with keyReference={cls.keyReference}") class Puk1(Puk): + name = 'PUK1' keyReference = 0x01 class Puk2(Puk): + name = 'PUK2' keyReference = 0x81 class Pin(DecimalHexParam): @@ -539,9 +545,11 @@ + f' {cls.get_name()} cannot find pinCode with keyReference={cls.keyReference}') class Pin1(Pin): + name = 'PIN1' keyReference = 0x01 class Pin2(Pin): + name = 'PIN2' keyReference = 0x81 @classmethod @@ -557,9 +565,11 @@ + f' {cls.get_name()} cannot find pinCode with keyReference={cls.keyReference} in {naa=}') class Adm1(Pin): + name = 'ADM1' keyReference = 0x0A class Adm2(Pin): + name = 'ADM2' keyReference = 0x0B class AlgoConfig(ConfigurableParameter): @@ -593,16 +603,19 @@ class K(BinaryParam, AlgoConfig): """use validate_val() from BinaryParam, and apply_val() from AlgoConfig""" + name = 'K' algo_config_key = 'key' allow_len = (128 // 8, 256 // 8) # length in bytes (from BinaryParam); TUAK also allows 256 bit class Opc(K): + name = 'OPc' algo_config_key = 'opc' class MilenageRotationConstants(BinaryParam, AlgoConfig): """rotation constants r1,r2,r3,r4,r5 of Milenage, Range 0..127. See 3GPP TS 35.206 Sections 2.3 + 5.3. Provided as octet-string concatenation of all 5 constants. Expects a bytes-like object of length 5, with each byte in the range of 0..127. The default value by 3GPP is '4000204060' (hex notation)""" + name = 'MilenageRotation' algo_config_key = 'rotationConstants' allow_len = 5 # length in bytes (from BinaryParam) @@ -625,11 +638,13 @@ 00000000000000000000000000000004 00000000000000000000000000000008 """ + name = 'MilenageXOR' algo_config_key = 'xoringConstants' allow_len = 80 # length in bytes (from BinaryParam) class TuakNumberOfKeccak(IntegerParam, AlgoConfig): """Number of iterations of Keccak-f[1600] permutation as recomended by Section 7.2 of 3GPP TS 35.231""" + name = 'KECCAK-N' algo_config_key = 'numberOfKeccak' min_val = 1 max_val = 255 -- To view, visit https://gerrit.osmocom.org/c/pysim/+/40093?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: I31f390d634e58c384589c50a33ca45d6f86d4e10 Gerrit-Change-Number: 40093 Gerrit-PatchSet: 12 Gerrit-Owner: neels <nhofmeyr(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: neels <nhofmeyr(a)sysmocom.de>

1 0

[S] Change in pysim[master]: personalization: set example input values
by laforge 30 Jan '26

30 Jan '26

laforge has submitted this change. ( https://gerrit.osmocom.org/c/pysim/+/40094?usp=email ) Change subject: personalization: set example input values ...................................................................... personalization: set example input values For all ConfigurableParameter subclasses, provide an example_input. This may be useful for downstream projects' user interaction, to suggest a value or prefill an input field, as appropriate. Related: SYS#6768 Change-Id: I2672fedcbc32cb7a6cb0c233a4a22112bd9aae03 --- M pySim/esim/saip/personalization.py 1 file changed, 17 insertions(+), 2 deletions(-) Approvals: laforge: Looks good to me, approved Jenkins Builder: Verified diff --git a/pySim/esim/saip/personalization.py b/pySim/esim/saip/personalization.py index 943784f..54aaa43 100644 --- a/pySim/esim/saip/personalization.py +++ b/pySim/esim/saip/personalization.py @@ -115,6 +115,7 @@ min_len = None max_len = None allow_len = None # a list of specific lengths + example_input = None def __init__(self, input_value=None): self.input_value = input_value # the raw input value as given by caller @@ -306,6 +307,7 @@ name = 'ICCID' min_len = 18 max_len = 20 + example_input = '998877665544332211' @classmethod def validate_val(cls, val): @@ -326,6 +328,7 @@ name = 'IMSI' min_len = 6 max_len = 15 + example_input = '00101' + ('0' * 10) @classmethod def apply_val(cls, pes: ProfileElementSequence, val): @@ -497,6 +500,7 @@ allow_len = 8 rpad = 16 keyReference = None + example_input = '0' * allow_len @classmethod def apply_val(cls, pes: ProfileElementSequence, val): @@ -523,6 +527,7 @@ rpad = 16 min_len = 4 max_len = 8 + example_input = '0' * max_len keyReference = None @staticmethod @@ -546,9 +551,10 @@ class Pin1(Pin): name = 'PIN1' + example_input = '0' * 4 # PIN are usually 4 digits keyReference = 0x01 -class Pin2(Pin): +class Pin2(Pin1): name = 'PIN2' keyReference = 0x81 @@ -568,7 +574,7 @@ name = 'ADM1' keyReference = 0x0A -class Adm2(Pin): +class Adm2(Adm1): name = 'ADM2' keyReference = 0x0B @@ -591,6 +597,7 @@ class AlgorithmID(DecimalParam, AlgoConfig): algo_config_key = 'algorithmID' allow_len = 1 + example_input = 1 # Milenage @classmethod def validate_val(cls, val): @@ -606,6 +613,7 @@ name = 'K' algo_config_key = 'key' allow_len = (128 // 8, 256 // 8) # length in bytes (from BinaryParam); TUAK also allows 256 bit + example_input = '00' * allow_len[0] class Opc(K): name = 'OPc' @@ -618,6 +626,7 @@ name = 'MilenageRotation' algo_config_key = 'rotationConstants' allow_len = 5 # length in bytes (from BinaryParam) + example_input = '0a 0b 0c 0d 0e' @classmethod def validate_val(cls, val): @@ -641,6 +650,11 @@ name = 'MilenageXOR' algo_config_key = 'xoringConstants' allow_len = 80 # length in bytes (from BinaryParam) + example_input = ('00000000000000000000000000000000' + ' 00000000000000000000000000000001' + ' 00000000000000000000000000000002' + ' 00000000000000000000000000000004' + ' 00000000000000000000000000000008') class TuakNumberOfKeccak(IntegerParam, AlgoConfig): """Number of iterations of Keccak-f[1600] permutation as recomended by Section 7.2 of 3GPP TS 35.231""" @@ -648,3 +662,4 @@ algo_config_key = 'numberOfKeccak' min_val = 1 max_val = 255 + example_input = '1' -- To view, visit https://gerrit.osmocom.org/c/pysim/+/40094?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: I2672fedcbc32cb7a6cb0c233a4a22112bd9aae03 Gerrit-Change-Number: 40094 Gerrit-PatchSet: 12 Gerrit-Owner: neels <nhofmeyr(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org>

1 0

2026

2025

2024

2023

2022

gerrit-log