BRANCHpmaier/fixgrEMAIL_NOTIFICATIONSStarted by user pmaier@sysmocom.depmaierpmaier@sysmocom.de00009879649879641.00002097c74d7fbd0d27907d599570270508e2e949b65903c74d7fbd0d27907d599570270508e2e949b65903refs/remotes/origin/pmaier/fixgrc74d7fbd0d27907d599570270508e2e949b65903c74d7fbd0d27907d599570270508e2e949b65903refs/remotes/origin/pmaier/fixgr2096f2567de387c71cf637a4cb1365089ba7173870b0f2567de387c71cf637a4cb1365089ba7173870b0origin/masterf2567de387c71cf637a4cb1365089ba7173870b0f2567de387c71cf637a4cb1365089ba7173870b0origin/master13208f289234f74045496c3752bf1bb9397302f3e9128f289234f74045496c3752bf1bb9397302f3e912refs/remotes/origin/pmaier/ara-m8f289234f74045496c3752bf1bb9397302f3e9128f289234f74045496c3752bf1bb9397302f3e912refs/remotes/origin/pmaier/ara-m13533d815859830197b9c55e0a49a1bd2532f71285e13d815859830197b9c55e0a49a1bd2532f71285e1refs/remotes/origin/pmaier/keepfiles3d815859830197b9c55e0a49a1bd2532f71285e13d815859830197b9c55e0a49a1bd2532f71285e1refs/remotes/origin/pmaier/keepfiles1421568e8ae9b817ec2473907bfdd30870410890ffb0568e8ae9b817ec2473907bfdd30870410890ffb0refs/remotes/origin/pmaier/fixapdu568e8ae9b817ec2473907bfdd30870410890ffb0568e8ae9b817ec2473907bfdd30870410890ffb0refs/remotes/origin/pmaier/fixapdu14676a555ac5ccad9ee2cae5e0ba247990f824f8e9b66a555ac5ccad9ee2cae5e0ba247990f824f8e9b6refs/remotes/origin/pmaier/aramtest6a555ac5ccad9ee2cae5e0ba247990f824f8e9b66a555ac5ccad9ee2cae5e0ba247990f824f8e9b6refs/remotes/origin/pmaier/aramtest1816e4ea1c997348370ba1d7415bfd926ac873b58c87e4ea1c997348370ba1d7415bfd926ac873b58c87refs/remotes/origin/osmith/wipe4ea1c997348370ba1d7415bfd926ac873b58c87e4ea1c997348370ba1d7415bfd926ac873b58c87refs/remotes/origin/osmith/wip2022f57f047c627392c40fbe349b1a73ccebe05dc005f57f047c627392c40fbe349b1a73ccebe05dc005refs/remotes/origin/pmaier/otatestf57f047c627392c40fbe349b1a73ccebe05dc005f57f047c627392c40fbe349b1a73ccebe05dc005refs/remotes/origin/pmaier/otatest4024f888a0414101dee4f20aa12579491fcc71707b34f888a0414101dee4f20aa12579491fcc71707b3refs/remotes/origin/master4f888a0414101dee4f20aa12579491fcc71707b34f888a0414101dee4f20aa12579491fcc71707b3refs/remotes/origin/masterc74d7fbd0d27907d599570270508e2e949b65903c74d7fbd0d27907d599570270508e2e949b65903refs/remotes/origin/pmaier/fixgrhttps://gerrit.osmocom.org/pysimfalse#2097987964988845master-pysim #20972097falsefalse209789066SUCCESS1776241586322https://jenkins.osmocom.org/jenkins/view/All%20no%20Gerrit/job/master-pysim/2097/- pySim/esim/saip/__init__.py3752aeb94e5bd4a25063eee28a11e0564e8cf65e1773160488000https://jenkins.osmocom.org/jenkins/user/laforgelaforgelaforge@osmocom.orgpySim.esim.saip.File: Support pinStatusTemplateDO + lcsi
The tuples defining a DF or ADF in an eSIM template must contain a
pinStatusTemplateDO. When parsing tuples into a File() instance, we
must save it, and re-create it at the time we re-encode that file.
Same applies to the lcsi (life cycle state indicator), which may
optionally exist for any file.
Change-Id: I073aa4374f2cd664d07fa0224bf0d4c809cdf4aa
Closes: OS#6955
2026-03-10 16:34:48 +00003752aeb94e5bd4a25063eee28a11e0564e8cf65epySim.esim.saip.File: Support pinStatusTemplateDO + lcsieditpySim/esim/saip/__init__.py
- pySim/cdma_ruim.pya5a5865c7c7fdeaf679cf20aa987a96780137eac1773164590000https://jenkins.osmocom.org/jenkins/user/laforgelaforgelaforge@osmocom.orgcdma_ruim: fix copy-pasted desc for EF.AD
Change-Id: I2338f35c21978dd6b8916c0abd57b94f5e087655
2026-03-10 17:43:10 +0000a5a5865c7c7fdeaf679cf20aa987a96780137eaccdma_ruim: fix copy-pasted desc for EF.ADeditpySim/cdma_ruim.py
- pySim/esim/http_json_api.py0634f77308d2674872cb96a7e70cf1a2e75df3eb1773742632000https://jenkins.osmocom.org/jenkins/user/pmaierpmaier@sysmocom.depmaier@sysmocom.deesim/http_json_api: allow URL rewriting
The URL used when HTTP requests are performed is defined statically
with the url_prefix passed to the constructor of JsonHttpApiClient
together with the path property in JsonHttpApiFunction.
For applications that require dynamic URLs there is no way to rewrite
the URL. Let's add a mechanism that allows API users to apply custom
URL reqriting rules by adding a rewrite_url method to
JsonHttpApiFunction. API users may then overload this method with a
custom implementation as needed.
Related: SYS#7918
Change-Id: Id2713a867079cc140517fe312189e5e2162608a5
2026-03-17 11:17:12 +01000634f77308d2674872cb96a7e70cf1a2e75df3ebesim/http_json_api: allow URL rewritingeditpySim/esim/http_json_api.py
- pySim/global_platform/__init__.pyea1d5af38337073e0d14d396540b2eef96e634b61774042379000https://jenkins.osmocom.org/jenkins/user/laforgelaforgelaforge@osmocom.orgglobal_platform: fix typo in SupportedTlsCipherSuitesForScp81
The attribute name is misspelled. The BER-TLV infrastructure looks
for `_construct`; this typo means `SupportedTlsCipherSuitesForScp81`
will never decode its content.
Change-Id: I0f637951b0eeb7eca2a8b543baa737f216a935ed
2026-03-20 14:32:59 -0700ea1d5af38337073e0d14d396540b2eef96e634b6global_platform: fix typo in SupportedTlsCipherSuitesForScp81editpySim/global_platform/__init__.py
- pySim/global_platform/__init__.py3b87ba3cba9d8cb8003c9ba8276a08145fb946f51774042379000https://jenkins.osmocom.org/jenkins/user/laforgelaforgelaforge@osmocom.orgglobal_platform: fix typo in ApplicationTemplate
The keyword argument should be `nested=`. As written `ApplicationAID`
is silently ignored - `ApplicationTemplate` will not descend into its
nested TLVs.
Change-Id: If45dbb0c9b09fe53560d109957ce339267a9f2b0
2026-03-20 14:32:59 -07003b87ba3cba9d8cb8003c9ba8276a08145fb946f5global_platform: fix typo in ApplicationTemplateeditpySim/global_platform/__init__.py
- pySim/global_platform/__init__.pyb76cc80ea167999a8eaa56f4c39d10e43aadd40b1774042379000https://jenkins.osmocom.org/jenkins/user/laforgelaforgelaforge@osmocom.orgglobal_platform: fix store_data() returning last chunk only
The loop builds up `response` across multiple STORE DATA blocks,
but the function returns only `data` - the response from the
*last* block. It should return the accumulated response instead.
Change-Id: I3e15c8004d1e366e8c3896e559656622f48bb1a2
2026-03-20 14:32:59 -0700b76cc80ea167999a8eaa56f4c39d10e43aadd40bglobal_platform: fix store_data() returning last chunk onlyeditpySim/global_platform/__init__.py
- pySim/global_platform/__init__.py3b50e64c8b7786804510d91b3ab90c9d8aef86d21774042379000https://jenkins.osmocom.org/jenkins/user/laforgelaforgelaforge@osmocom.orgglobal_platform: fix s/GET/STORE/ DATA in docs
Both `do_store_data` and `store_data` have identical docstrings that
incorrectly describe the command as GET DATA. Should be "STORE DATA".
Take a chance to fix missing space between `v2.3` and `Section`.
Change-Id: I33fc80ab8ca50fadc38217b0005eec6169c8e34e
2026-03-20 14:32:59 -07003b50e64c8b7786804510d91b3ab90c9d8aef86d2global_platform: fix s/GET/STORE/ DATA in docseditpySim/global_platform/__init__.py
- pySim/global_platform/scp.py209d13e23376992a3112f021d26bb38704a3ed031774042379000https://jenkins.osmocom.org/jenkins/user/laforgelaforgelaforge@osmocom.orgglobal_platform: fix docstring for Scp03SessionKeys._get_icv()
Change-Id: I8983bc27f581295544360ba8b4ae1d28b3ea850f
2026-03-20 14:32:59 -0700209d13e23376992a3112f021d26bb38704a3ed03global_platform: fix docstring for Scp03SessionKeys._get_icv()editpySim/global_platform/scp.py
- pySim/ara_m.py14ec52a06c6cfe84bc578010fa7f8b0d9ba167ed1774042379000https://jenkins.osmocom.org/jenkins/user/laforgelaforgelaforge@osmocom.orgara_m: fix exceptions not being raised properly
Exceptions are meant to be thrown/raised, not returned.
Change-Id: Id799c264447e22887edcd2dc7eb991cf0af1bbfc
2026-03-20 14:32:59 -070014ec52a06c6cfe84bc578010fa7f8b0d9ba167edara_m: fix exceptions not being raised properlyeditpySim/ara_m.py
- pySim/ara_m.py83bfdc0d3bba437437346c439df5ba5c4d01711e1774042379000https://jenkins.osmocom.org/jenkins/user/laforgelaforgelaforge@osmocom.orgara_m: fix undefined variable used in a format-string
Change-Id: I310a5d461bae2b5e4d8e07097000b079c23aa0f6
2026-03-20 14:32:59 -070083bfdc0d3bba437437346c439df5ba5c4d01711eara_m: fix undefined variable used in a format-stringeditpySim/ara_m.py
- pySim/sms.pyd32bce19f631026d0b621f09c3a540a3b1779e481774042379000https://jenkins.osmocom.org/jenkins/user/laforgelaforgelaforge@osmocom.orgsms: fix flags_construct in SMS_DELIVER
* field `tp_rp` appears at bit positions 7 and 5
** bit 7 should be `tp_rp` (Reply Path)
** bit 5 should be `tp_sri` (Status Report Indication)
* field `tp_lp` is completely missing
** should be at bit position 3
Change-Id: I0274849f0fa07281b5e050af429ffda7d249f9e8
2026-03-20 14:32:59 -0700d32bce19f631026d0b621f09c3a540a3b1779e48sms: fix flags_construct in SMS_DELIVEReditpySim/sms.py
- pySim/ota.py09ae327f8b53e325fb7ea799dd09b120a3c0acb41774042379000https://jenkins.osmocom.org/jenkins/user/laforgelaforgelaforge@osmocom.orgota: OtaAlgo{Crypt,Auth}: fix algo_auth vs algo_crypt
* OtaAlgoCrypt.from_keyset() searches by `otak.algo_crypt`
but the error message prints `otak.algo_auth`. Should be
`otak.algo_crypt` instead.
* OtaAlgoAuth.__init__() checks `algo_auth` but the error message
prints `algo_crypt`. Should be `otak.algo_auth` instead.
Change-Id: Ia636fffaeadc68e3f6d5b65d477e753834c95895
2026-03-20 14:32:59 -070009ae327f8b53e325fb7ea799dd09b120a3c0acb4ota: OtaAlgo{Crypt,Auth}: fix algo_auth vs algo_crypteditpySim/ota.py
- pySim/utils.py24127e985a8388bfb38124a6a51486f774aec3ba1774042379000https://jenkins.osmocom.org/jenkins/user/laforgelaforgelaforge@osmocom.orgutils: dec_plmn(): remove redundant call
Change-Id: Ic95c3992ed57eb8fee952ec2dc7f092dd7689579
2026-03-20 14:32:59 -070024127e985a8388bfb38124a6a51486f774aec3bautils: dec_plmn(): remove redundant calleditpySim/utils.py
- pySim/utils.py5b2fabde62690359198f60be2f04e7a3ee45f6e21774042379000https://jenkins.osmocom.org/jenkins/user/laforgelaforgelaforge@osmocom.orgutils: DataObjectCollection.encode(): fix TypeError
`members_by_name` is a plain dictionary. Calling it with `()` raises:
TypeError: 'dict' object is not callable
Change-Id: I7e0c09aa7303f1506fe3a025fdc3779919dd0e6c
2026-03-20 14:32:59 -07005b2fabde62690359198f60be2f04e7a3ee45f6e2utils: DataObjectCollection.encode(): fix TypeErroreditpySim/utils.py
- pySim/ts_51_011.py5138208ee6b65fb70f4ecaee1aa5e529f9468c201774042379000https://jenkins.osmocom.org/jenkins/user/laforgelaforgelaforge@osmocom.orgts_51_011: EF.EXT[6-7]: fix typo in desc
Change-Id: I93df1c9fd8a4d588ed7ed19ec2dc1d304412fc3d
2026-03-20 14:32:59 -07005138208ee6b65fb70f4ecaee1aa5e529f9468c20ts_51_011: EF.EXT[6-7]: fix typo in desceditpySim/ts_51_011.py
- pySim/ts_31_102.py11a7a7e3b1e040c9213a251eb8543f21477f26ac1774042379000https://jenkins.osmocom.org/jenkins/user/laforgelaforgelaforge@osmocom.orgts_31_102: fix description for EF_5GS3GPPLOCI
Change-Id: I9cf3adfce65090fedb3f0fd33c9b3d15a2c5fb8c
2026-03-20 14:32:59 -070011a7a7e3b1e040c9213a251eb8543f21477f26acts_31_102: fix description for EF_5GS3GPPLOCIeditpySim/ts_31_102.py
- pySim/ts_31_102.pycaddd1c7a0ef549bdba4d8b31287232b29fbd0121774042379000https://jenkins.osmocom.org/jenkins/user/laforgelaforgelaforge@osmocom.orgts_31_102: EF_5G_PROSE_UIR: fix copy-pasted inner class name
Change-Id: I460e5ad70f35026d0d794271a4aef17323c14dfb
2026-03-20 14:32:59 -0700caddd1c7a0ef549bdba4d8b31287232b29fbd012ts_31_102: EF_5G_PROSE_UIR: fix copy-pasted inner class nameeditpySim/ts_31_102.py
- pySim/cdma_ruim.py99aef1fecfe7dae70a33fd597e996f5c2ecc92901774042379000https://jenkins.osmocom.org/jenkins/user/laforgelaforgelaforge@osmocom.orgcdma_ruim: fix inaccurate comment for EF_AD
Change-Id: I71ea27fd30e44685ff35f49843072ca392995973
2026-03-20 14:32:59 -070099aef1fecfe7dae70a33fd597e996f5c2ecc9290cdma_ruim: fix inaccurate comment for EF_ADeditpySim/cdma_ruim.py
- pySim/esim/saip/__init__.py115b517c6a67f31b668cb7526577b02d5c6390381774042379000https://jenkins.osmocom.org/jenkins/user/laforgelaforgelaforge@osmocom.orgesim/saip: raise an exception properly
Change-Id: Ia3749c02120fdc16e556214d0461cbeca032447b
2026-03-20 14:32:59 -0700115b517c6a67f31b668cb7526577b02d5c639038esim/saip: raise an exception properlyeditpySim/esim/saip/__init__.py
- pySim/ts_51_011.pyb59363b49e5a86021e96358526440d85ee88c1f21774280678000https://jenkins.osmocom.org/jenkins/user/pmaierpmaier@sysmocom.depmaier@sysmocom.depySim/EF.SMSP: remove superflous line break
Change-Id: Ie02e02546e708e2c339810812188bd8e8af2a720
2026-03-23 16:44:38 +0100b59363b49e5a86021e96358526440d85ee88c1f2pySim/EF.SMSP: remove superflous line breakeditpySim/ts_51_011.py
- pySim/transport/pcsc.py473f31066c97ec3256ff9e2c15d55a2a544fd2861774292599000https://jenkins.osmocom.org/jenkins/user/laforgelaforgelaforge@osmocom.orgpySim/pcsc/cosmetic: reformat comment
Change-Id: Ic04bdfbc6727cc670679c377c1afd1de53504b8f
2026-03-23 19:03:19 +0000473f31066c97ec3256ff9e2c15d55a2a544fd286pySim/pcsc/cosmetic: reformat commenteditpySim/transport/pcsc.py
- pySim/ts_51_011.py2ef9abf23e05481fdeab43cb6e231a368a321b1d1774367724000https://jenkins.osmocom.org/jenkins/user/laforgelaforgelaforge@osmocom.orgpySim/EF.SMSP: add an additional de_encode test for EF_SMSP
Let's add another testvector where we test what happens when we populate
none of the fields except for the tp_sc_addr.
Related: SYS#7765
Change-Id: I12b600ab17d1acfdddaffe6006095acf1a4228c9
2026-03-24 15:55:24 +00002ef9abf23e05481fdeab43cb6e231a368a321b1dpySim/EF.SMSP: add an additional de_encode test for EF_SMSPeditpySim/ts_51_011.py
- pySim/ts_51_011.py969f9c0e4bcb3cbf46be3a592315aa8a9a39e8071774449296000https://jenkins.osmocom.org/jenkins/user/pmaierpmaier@sysmocom.depmaier@sysmocom.depySim/EF.SMSP: fix encoding of TP-Destination Address
The TP-Destination Address in EF.SMSP uses the same encoding as the
TS-Service Centre Address field. However, even though the encoding
of both fields looks almost identical, it actually isn't.
The TS-Service Centre Address field encodes the length field as
octets required for the call_number + one octet for ton_npi.
(see also: 3GPP TS 24.011, section 8.2.5.2)
The TP-Destination Address uses the number of digits of the
call_number directly in the length field.
(see also: 3GPP TS 23.040, section 9.1.2.5)
Related: SYS#7765
Change-Id: I55c123c9e244e5a6e71a0348f5d476ef03e618e8
2026-03-25 15:34:56 +0100969f9c0e4bcb3cbf46be3a592315aa8a9a39e807pySim/EF.SMSP: fix encoding of TP-Destination AddresseditpySim/ts_51_011.py
- pySim/global_platform/__init__.pyc7e68e1281db0fbf0e456e7807e642cb5ee49c3a1774461930000https://jenkins.osmocom.org/jenkins/user/laforgelaforgelaforge@osmocom.orgglobal_platform: install_cap_parser: argument groups cannot be nested
pySim-shell currently does not work on systems with Python 3.14+:
File ".../pysim/pySim/global_platform/__init__.py", line 868, in AddlShellCommands
install_cap_parser_inst_prm_g_grp = install_cap_parser_inst_prm_g.add_argument_group()
File "/usr/lib/python3.14/argparse.py", line 1794, in add_argument_group
raise ValueError('argument groups cannot be nested')
ValueError('argument groups cannot be nested')
The problem is that install_cap_parser creates a nested group inside
of mutually exclusive group. argparse never supported group nesting
properly, so it has been deprecated since Python 3.11, and eventually
got removed in Python 3.14.
Remove group nesting, adjust the usage string, and implement the
mutual exclusiveness enforcement manually in do_install_cap().
Change-Id: Idddf72d5a745345e134b23f2f01e0257d0667579
2026-03-25 18:05:30 +0000c7e68e1281db0fbf0e456e7807e642cb5ee49c3aglobal_platform: install_cap_parser: argument groups cannot be nestededitpySim/global_platform/__init__.py
- tests/unittests/test_globalplatform.pypySim/global_platform/install_param.pyd9d62ee72929fca356fd3e751e2a3284e84475ac1774461930000https://jenkins.osmocom.org/jenkins/user/laforgelaforgelaforge@osmocom.orgglobal_platform: refactor gen_install_parameters()
gen_install_parameters() had contradictory logic: the outer guard
required all three arguments to be non-None/non-empty (making them
mutually inclusive), while the inner checks then treated each one
as optional.
Make each parameter independently optional (defaulting to None) and
remove the all-or-nothing check. Simplify the function body to a
straightforward single-pass construction of system_specific_params.
Change-Id: I8756fb38016cdf0527fe2e21edb44381d1dc557f
2026-03-25 18:05:30 +0000d9d62ee72929fca356fd3e751e2a3284e84475acglobal_platform: refactor gen_install_parameters()edittests/unittests/test_globalplatform.pyeditpySim/global_platform/install_param.py
- docs/put_key-tutorial.rstdocs/shell.rst8c81e2cdf9bb1f8d831e817379cb0d03347df46b1774462052000https://jenkins.osmocom.org/jenkins/user/laforgelaforgelaforge@osmocom.orgdocs/put_key: add tutorial that explains how to manage global platform keys
With the increased interest in using GlobalPlatform features of
UICC and eUICCs (OTA-SMS, applets, etc.), also comes an increased
interest in how the related GlobalPlatform keys can be managed
(key rotation, adding/removing keysets from/to a Security Domain).
Unfortunately, many aspects of this topic are not immediately
obvious for the average user. Let's add a tutorial that contains
some practical examples to shine some light on the topic.
Related: SYS#7881
Change-Id: I163dfedca3df572cb8442e9a4a280e6c5b00327e
2026-03-25 18:07:32 +00008c81e2cdf9bb1f8d831e817379cb0d03347df46bdocs/put_key: add tutorial that explains how to manage global platformeditdocs/shell.rstadddocs/put_key-tutorial.rst
- pySim/transport/__init__.pypySim-shell.pyadd4b991b78d96a6790541d28bc3ba52e1c619e51774866373000https://jenkins.osmocom.org/jenkins/user/laforgelaforgelaforge@osmocom.orgtransport: change APDU format paradigm
Unfortunately we have mixed up the concept of TPDUs and APDUs in
earlier versions of pySim-shell. This lead to problems with
detecteding the APDU case properly (see also ISO/IEC 7816-3) and
also prevented us from adding support for T=1.
This problem has been fixed long time ago and all APDUs sent from
the pySim-shell code should be well formed and valid according to
ISO/IEC 7816-3.
To ensure that we continue to format APDUs correctly as APDUs (and
not TPDUs) we have added a mechanism to the LinkBase class that
would either raise an exception or print a warning if someone
mistakenly tries to send an APDU that is really a TPDU. Whether a
warning is printed or an exception is raised is controlled via the
apdu_strict member in the LinkBase class, which is false (print
warning only) by default.
The reason why we have implemneted the mechanism this way was
because we wanted to ensure that existing APDU scripts (pySim-shell
apdu command) keep working, even though when those scripts uses
APDUs which are formally invalid.
Sending a TPDU instead of an APDU via a T=0 link will still work
in almost all cases. This is also the reason why this problem
slipped through unnoticed for long time. However, there may still
be subtile problems araising from this practice. The root of the
problem is that it is impossible to distinguish between APDU case
3 and 4 when a TPDU instead of an APDU is sent. However in order
to handle a case 4 APDU correctly we must be able to distinguish
the APDU case correctly to handle the case correctly.
ETSI TS 102 221, section 7.3.1.1.4, clause 4 is very clear about
the fact that not (only) the status word (e.g. 61xx) but the
APDU case is what matters.
To complete the logic in LinkBaseTpdu and to maintain compatibility
(older APDU scripts), we must still be able to switch between the
'apdu_strict' mode and the non-strict mode. However, since
pySim-shell, pySim-prog and pySim-read internally use proper APDUs,
we may enable the 'apdu_strict' mode by default.
At the same time we will limit the effect of pySim-shell's
apdu_strict setable to the apdu command only. By doing so, the
bahviour of the apdu command is not altered. Users will still
have to enable the 'strict' mode explicitly. At the same time
all the internal functionality of pySim-shell will always use
the 'strict' mode.
Related: OS#6970
Change-Id: I9a531a825def318b28bf58291d811cf119003fab
2026-03-30 10:26:13 +0000add4b991b78d96a6790541d28bc3ba52e1c619e5transport: change APDU format paradigmeditpySim/transport/__init__.pyeditpySim-shell.py
- docs/card-key-provider.rst74ac191ae60fcdffe39bc98d46fc94a315a879321774945567000https://jenkins.osmocom.org/jenkins/user/laforgelaforgelaforge@osmocom.orgdocs/card-key-provider: fix heading levels and typo
The "ADM PIN" and "SCP02 / SCP03" sub-sections of "Field naming" used
the '~' heading character, which Sphinx resolved to level 4 - skipping
level 3 and throwing build ERRORs. As a result, both sub-sections
had no heading at all. Change both to '^' (level 3) to match the
other sub-sections in this file.
While at it, fix a typo: "consisting if" -> "consisting of".
Change-Id: Ia56efc7fadcc0fd62e87e63850b929d2f80851ba
2026-03-31 08:26:07 +000074ac191ae60fcdffe39bc98d46fc94a315a87932docs/card-key-provider: fix heading levels and typoeditdocs/card-key-provider.rst
- docs/shell.rstb42d417bbefc77f48d946854f290ee9cb02dfe5d1774945567000https://jenkins.osmocom.org/jenkins/user/laforgelaforgelaforge@osmocom.orgdocs/shell: fix copy-paste typos in editor command descriptions
Two adjacent commands (`edit_record_decoded`, `edit_binary_decoded`)
had identical copy-pasted error messages with three typos each:
"modificatiosn" -> "modifications"
"us the" -> "use the"
"comamdn" -> "command"
Change-Id: Ie23baba4634e2cc40f81439fb11b102778aed1f6
2026-03-31 08:26:07 +0000b42d417bbefc77f48d946854f290ee9cb02dfe5ddocs/shell: fix copy-paste typos in editor command descriptionseditdocs/shell.rst
- docs/saip-tool.rst4215a3bfd3762f1ccee788d1f205e43a2460b8bc1774945567000https://jenkins.osmocom.org/jenkins/user/laforgelaforgelaforge@osmocom.orgdocs/saip-tool: fix typo "insertaion" -> "insertion"
Change-Id: Ie9c9235ec964a15fab19d6ca5a83b2b1ddf07e7b
2026-03-31 08:26:07 +00004215a3bfd3762f1ccee788d1f205e43a2460b8bcdocs/saip-tool: fix typo "insertaion" -> "insertion"editdocs/saip-tool.rst
- docs/legacy.rst2cfb0972df36214d85b18d1071ed6b657020911c1774945567000https://jenkins.osmocom.org/jenkins/user/laforgelaforgelaforge@osmocom.orgdocs/legacy: fix typo "EF,IMSI" -> "EF.IMSI"
Change-Id: I1f246ec008a57b2373ed3f5531ab4166101f4dd0
2026-03-31 08:26:07 +00002cfb0972df36214d85b18d1071ed6b657020911cdocs/legacy: fix typo "EF,IMSI" -> "EF.IMSI"editdocs/legacy.rst
- docs/put_key-tutorial.rstf7b86e1920cbd4f378bfe81fb51ad5d47c2a0c3d1774945567000https://jenkins.osmocom.org/jenkins/user/laforgelaforgelaforge@osmocom.orgdocs/put_key-tutorial: fix three typos
"verifiation" -> "verification"
"this is identifies" -> "this identifies" (extra word)
"and and" -> "and" (doubled word)
Change-Id: I1ae6b01638cc2c3dd8355ba801f85cc179ca8bd3
2026-03-31 08:26:07 +0000f7b86e1920cbd4f378bfe81fb51ad5d47c2a0c3ddocs/put_key-tutorial: fix three typoseditdocs/put_key-tutorial.rst
- pySim-read.pya1d3b8f5e887fc72fefae24248aa1fd4ffbe90941774950388000https://jenkins.osmocom.org/jenkins/user/pmaierpmaier@sysmocom.depmaier@sysmocom.depySim-read: remove import random
In pySim-read we do not have to compute any random numbers, so
we may remove random from the imports
Change-Id: Iae4ee6aafb339cc682345299b92b4ecd0bbca14e
2026-03-31 11:46:28 +0200a1d3b8f5e887fc72fefae24248aa1fd4ffbe9094pySim-read: remove import randomeditpySim-read.py
- contrib/csv-to-pgsql.pypySim/log.pypySim-shell.pyee06ab987fc29640fd59b6b0696596e8a4eb3f9a1774956586000https://jenkins.osmocom.org/jenkins/user/laforgelaforgelaforge@osmocom.orgPySimLogger: add parameter to set initial log-level/verbosity
When we initialize a new PySimLogger, we always call the setup method
first and then use the set_verbose and set_level method to configure
the initial log level and the initial log verbosity. However, we
initialize the PySimLogger in all our programs the same way and we
end up with the same boilerplate code every time. Let's add a keyword
parameter to the setup method where we can pass our opts.verbose (bool)
parameter so that the setup method can do the work for the main program.
In case the caller wants a different default configuration he still can
call set_verbose and set_level methods as needed.
Change-Id: I4b8ef1e203186878910c9614a1d900d5759236a8
2026-03-31 11:29:46 +0000ee06ab987fc29640fd59b6b0696596e8a4eb3f9aPySimLogger: add parameter to set initial log-level/verbosityeditpySim/log.pyeditpySim-shell.pyeditcontrib/csv-to-pgsql.py
- pySim-shell.pyc995bb1ec230b88d0b8891d78cfdb30d34fcf4e21774956586000https://jenkins.osmocom.org/jenkins/user/laforgelaforgelaforge@osmocom.orgpySim-shell/cosmetic: remove semicolon
Change-Id: I629bacd432491211b939fcd2bed554b44ef441bc
2026-03-31 11:29:46 +0000c995bb1ec230b88d0b8891d78cfdb30d34fcf4e2pySim-shell/cosmetic: remove semicoloneditpySim-shell.py
- tests/pySim-prog_test/sysmoISIM-SJA2.oktests/pySim-prog_test/sysmosim-gr1.oktests/pySim-prog_test/Wavemobile-SIM.oktests/pySim-prog_test/sysmoISIM-SJA5.oktests/pySim-prog_test/sysmoUSIM-SJS1.okpySim-prog.pytests/pySim-prog_test/Fairwaves-SIM.okpySim-read.pytests/pySim-prog_test/fakemagicsim.okca8fada7b6d5b36fab87cab66f8b75ed180a8ca21774956586000https://jenkins.osmocom.org/jenkins/user/laforgelaforgelaforge@osmocom.orgpySim-prog/pySim-read: add pySimLogger and verbose cmdline argument
pySim-prog and pySim-read do not integrate the pySimLogger yet. As we
may add more debug output that should not be visible on normal use, we
should ensure that the pySimLogger is correctly set up.
Change-Id: Ia2fa535fd9ce4ffa301c3f5d6f98c1f7a4716c74
2026-03-31 11:29:46 +0000ca8fada7b6d5b36fab87cab66f8b75ed180a8ca2pySim-prog/pySim-read: add pySimLogger and verbose cmdline argumentedittests/pySim-prog_test/fakemagicsim.okedittests/pySim-prog_test/sysmoISIM-SJA2.okedittests/pySim-prog_test/Wavemobile-SIM.okeditpySim-read.pyedittests/pySim-prog_test/Fairwaves-SIM.okedittests/pySim-prog_test/sysmosim-gr1.okedittests/pySim-prog_test/sysmoISIM-SJA5.okedittests/pySim-prog_test/sysmoUSIM-SJS1.okeditpySim-prog.py
- docs/conf.pya786590906996fe282b95b7d5e781f7d5a2625a01774959433000https://jenkins.osmocom.org/jenkins/user/fixeriaVadim Yanitskiyvyanitskiy@sysmocom.dedocs/conf.py: silence autosectionlabel duplicate-label warnings
sphinxarg.ext generates generic sub-headings ("Named arguments",
"Positional arguments", "Sub-commands", "General options", ...) for
every argparse command and tool. These repeat across many files and
trigger large numbers of autosectionlabel duplicate-label warnings.
Two-pronged fix:
* `autosectionlabel_maxdepth = 3` eliminates the depth-4+ warnings
(sub-headings inside each individual command block).
* `suppress_warnings` per file silences the residual depth-3 collisions
("serial reader", "decode_hex", "sub-commands", ...) that still
appear across tool documentation files.
Cross-references into these generic argparse-generated sections are not
a supported use-case, so suppressing the warnings is appropriate.
Change-Id: I9cdf2a4f6cbd435b16b90ab668205600ffd7c3b0
2026-03-31 19:17:13 +0700a786590906996fe282b95b7d5e781f7d5a2625a0docs/conf.py: silence autosectionlabel duplicate-label warningseditdocs/conf.py
- docs/conf.py19245d0d8bce1e7d87a5a5af328f9cacb2dc66001774959433000https://jenkins.osmocom.org/jenkins/user/fixeriaVadim Yanitskiyvyanitskiy@sysmocom.dedocs/conf.py: add autodoc_mock_imports for klein and twisted
The eSIM SM-DP+ server modules (`pySim.esim.es2p`, `pySim.esim.es9p`,
`pySim.esim.http_json_api`) unconditionally import optional server-side
dependencies at module level:
pySim.esim.es2p -- from klein import Klein
pySim.esim.http_json_api -- from twisted.web.server import Request
Both imports fail during a docs build if the packages are absent or
broken, causing three "autodoc: failed to import" warnings and three
missing chapters in the generated manual.
Even when klein and twisted are installed, twisted 23.10.0 (the
version pulled in transitively by smpp.twisted3's `Twisted~=23.10.0`
constraint) is incompatible with Python 3.13+ because twisted.web.http
unconditionally executes `import cgi`, a module that was removed from
the standard library in Python 3.13.
Fix: add `autodoc_mock_imports = ['klein', 'twisted']` to conf.py.
Sphinx inserts mock entries into sys.modules before each autodoc import
attempt, so the modules can be imported and documented without requiring
the real packages to be importable at build time.
Change-Id: I71650466f02a6a6d150650deed167c05d2cb6e64
2026-03-31 19:17:13 +070019245d0d8bce1e7d87a5a5af328f9cacb2dc6600docs/conf.py: add autodoc_mock_imports for klein and twistededitdocs/conf.py
- pySim/transport/pcsc.pyd5aa963caa89ea804aa86a098c6ccc41d30e28401775127210000https://jenkins.osmocom.org/jenkins/user/pmaierpmaier@sysmocom.depmaier@sysmocom.depysim/pcsc: do not use getProtocol for protocol selection
The documentation of the getProtocol provided by pyscard says:
"Return bit mask for the protocol of connection, or None if no
protocol set. The return value is a bit mask of
CardConnection.T0_protocol, CardConnection.T1_protocol,
CardConnection.RAW_protocol, CardConnection.T15_protocol"
This suggests that the purpose of getProtocol is not to determine
which protocols are supported. Its purpose is to determine which
protocol is currently selected (either through auto selection or
through the explicit selection made by the API user). This means
we are using getProtocol wrong.
So far this was no problem, since the auto-selected protocol
should be a supported protocol anyway. However, the automatic
protocol selection may not always return a correct result (see
bug report from THD-siegfried [1]).
Let's not trust the automatic protocol selection. Instead let's
parse the ATR and make the decision based on the TD1/TD2 bytes).
[1] https://osmocom.org/issues/6952
Related: OS#6952
Change-Id: Ib119948aa68c430e42ac84daec8b9bd542db7963
2026-04-02 12:53:30 +0200d5aa963caa89ea804aa86a098c6ccc41d30e2840pysim/pcsc: do not use getProtocol for protocol selectioneditpySim/transport/pcsc.py
- docs/index.rst.gitignoredocs/pysim_fs_sphinx.pydocs/conf.pytests/unittests/test_fs_coverage.pyfc932a2ee9a4dbecb4eec9fcdb97d95cae78708b1775575933000https://jenkins.osmocom.org/jenkins/user/laforgelaforgelaforge@osmocom.orgdocs: auto-generate Card Filesystem Reference
Add a Sphinx extension (docs/pysim_fs_sphinx.py) that hooks into the
builder-inited event and generates docs/filesystem.rst before Sphinx
reads any source files.
The generated page contains a hierarchical listing of all implemented
EFs and DFs, organised by application/specification (UICC/TS 102 221,
ADF.USIM/TS 31.102, ADF.ISIM/TS 31.103, SIM/TS 51.011). For each file,
the class docstring and any _test_de_encode / _test_decode vectors
are included as an encoding/decoding example table.
docs/filesystem.rst is fully generated at build time and is therefore
added to .gitignore.
Add tests/unittests/test_fs_coverage.py that walks all pySim.* modules
and verifies that every CardProfile, CardApplication, and standalone
CardDF subclass with EF/DF children is either listed in the SECTIONS
(and will appear in the docs) or explicitly EXCLUDED.
Change-Id: I06ddeefc6c11e04d7c24e116f3f39c8a6635856f
Related: OS#6316
2026-04-07 15:32:13 +0000fc932a2ee9a4dbecb4eec9fcdb97d95cae78708bdocs: auto-generate Card Filesystem Referenceedit.gitignoreadddocs/pysim_fs_sphinx.pyeditdocs/conf.pyeditdocs/index.rstaddtests/unittests/test_fs_coverage.py
- pySim/filesystem.py5e2fd148f89457e9fa83061aeb45cd59959c77351775575933000https://jenkins.osmocom.org/jenkins/user/laforgelaforgelaforge@osmocom.orgfilesystem: edit_{binary,record}_decoded: add encode/decode examples
When invoking `edit_binary_decoded` or `edit_record_decoded`, the
temp file opened in the editor now contains the EF's encode/decode
test vectors as //-comment lines below the JSON content, similar to
how 'git commit' appends comments to the commit message template.
The comment block is stripped before JSON parsing on save,
so it has no effect on the written data.
The feature is implemented via a new module-level JsonEditor context
manager class that encapsulates the full edit cycle:
* write JSON + examples to a TemporaryDirectory
* invoke the editor
* read back, strip //-comments, parse and return the result
Change-Id: I5a046a9c7ba7e08a98cf643d5a26bc669539b38f
Related: OS#6900
2026-04-07 15:32:13 +00005e2fd148f89457e9fa83061aeb45cd59959c7735filesystem: edit_{binary,record}_decoded: add encode/decode exampleseditpySim/filesystem.py
- pySim/filesystem.py5828c92c6693414301639ab5b84dc75529fe27ab1775576842000https://jenkins.osmocom.org/jenkins/user/fixeriaVadim Yanitskiyvyanitskiy@sysmocom.defilesystem: JsonEditor: use NamedTemporaryFile
A plain NamedTemporaryFile is sufficient here: we only need a single
file, not a directory to hold it. Using NamedTemporaryFile is simpler
(no subdirectory to manage) and gives us a .json suffix for free,
which editors use for syntax highlighting.
Change-Id: If3b0bd0fcc90732407dbd03b9cc883f7abeb948e
2026-04-07 22:47:22 +07005828c92c6693414301639ab5b84dc75529fe27abfilesystem: JsonEditor: use NamedTemporaryFileeditpySim/filesystem.py
- pySim/filesystem.py45220e00d567533cea4165d8e04994d8ef2ce3651775576842000https://jenkins.osmocom.org/jenkins/user/fixeriaVadim Yanitskiyvyanitskiy@sysmocom.defilesystem: JsonEditor: offer interactive retry on error
When json.loads() fails (e.g. the user made a syntax mistake), prompt
the user with "Re-open file for editing? [y]es/[n]o:" and loop back to
the editor if they answer 'y' or 'yes'. If the user declines, return
the original unmodified value so no write is attempted; the temp file
is still cleaned up by __exit__() in that case.
Change-Id: I9161b7becea0d8dfd3f5f740fbb253da2f061a1d
Related: OS#6899
2026-04-07 22:47:22 +070045220e00d567533cea4165d8e04994d8ef2ce365filesystem: JsonEditor: offer interactive retry on erroreditpySim/filesystem.py
- pySim/ts_51_011.pytests/unittests/test_files.py6b5fa38f147c41673bc89456984d0fe28c4c963b1775576842000https://jenkins.osmocom.org/jenkins/user/fixeriaVadim Yanitskiyvyanitskiy@sysmocom.detests: fix TransRecEF _test_de_encode to operate at file level
Previously, _test_de_encode vectors for TransRecEF subclasses were tested
via decode_record_hex()/encode_record_hex(), i.e. one record at a time,
with the decoded value being a scalar.
Switch test_de_encode_record() in TransRecEF_Test to use decode_hex() /
encode_hex() instead, so that vectors represent whole-file content
(decoded value is a list of records) -- consistent with how LinFixedEF
handles _test_de_encode. Update all existing vectors accordingly.
Change-Id: I4a9610f9ee39833cd0c90f64f89f5fbdd6f0846d
2026-04-07 22:47:22 +07006b5fa38f147c41673bc89456984d0fe28c4c963btests: fix TransRecEF _test_de_encode to operate at file leveleditpySim/ts_51_011.pyedittests/unittests/test_files.py
- pySim/ts_51_011.pyf2567de387c71cf637a4cb1365089ba7173870b01775576842000https://jenkins.osmocom.org/jenkins/user/fixeriaVadim Yanitskiyvyanitskiy@sysmocom.depySim/ts_51_011.py: add multi-record test vector for EF_PL
Change-Id: I9f7a444b18056b1683cbd52a25af950125531746
2026-04-07 22:47:22 +0700f2567de387c71cf637a4cb1365089ba7173870b0pySim/ts_51_011.py: add multi-record test vector for EF_PLeditpySim/ts_51_011.py
- pySim/transport/__init__.pytests/pySim-shell_test/apdu/test_apdu.scriptd7f4d204716176a62c48c4c73cb6a884a873d3101775750137000https://jenkins.osmocom.org/jenkins/user/pmaierpmaier@sysmocom.depmaier@sysmocom.depySim/transport: fix GET RESPONSE behaviour
The current behavior we implement in the method __send_apdu_T0 is
incomplete. Some details discussed in ETSI TS 102 221,
section 7.3.1.1.4, clause 4 seem to be not fully implemented. We
may also end up sending a GET RESPONSE in other APDU cases than
case 4 (the only case that uses the GET RESPONSE command).
Related: OS#6970
Change-Id: I26f0566af0cdd61dcc97f5f502479dc76adc37cc
2026-04-09 17:55:37 +0200d7f4d204716176a62c48c4c73cb6a884a873d310pySim/transport: fix GET RESPONSE behaviouredittests/pySim-shell_test/apdu/test_apdu.scripteditpySim/transport/__init__.py
- pySim/transport/__init__.pyc74d7fbd0d27907d599570270508e2e949b659031776241506000https://jenkins.osmocom.org/jenkins/user/pmaierpmaier@sysmocom.depmaier@sysmocom.defix
Change-Id: I1af2fd04ba935fc70a2306342eeb247d941c67af
2026-04-15 10:25:06 +0200c74d7fbd0d27907d599570270508e2e949b65903fixeditpySim/transport/__init__.py
githttps://jenkins.osmocom.org/jenkins/user/fixeriaVadim Yanitskiyhttps://jenkins.osmocom.org/jenkins/user/laforgelaforgehttps://jenkins.osmocom.org/jenkins/user/pmaierpmaier@sysmocom.de2097https://jenkins.osmocom.org/jenkins/view/All%20no%20Gerrit/job/master-pysim/JOB_TYPE=docs,a1=default,a3=default,a4=default,label=osmocom-master/2097/2097https://jenkins.osmocom.org/jenkins/view/All%20no%20Gerrit/job/master-pysim/JOB_TYPE=pylint,a1=default,a3=default,a4=default,label=osmocom-master/2097/2097https://jenkins.osmocom.org/jenkins/view/All%20no%20Gerrit/job/master-pysim/JOB_TYPE=test,a1=default,a3=default,a4=default,label=simtester/2097/