laforge has submitted this change. ( https://gerrit.osmocom.org/c/pysim/+/38274?usp=email )

Change subject: pySim.euicc: Fix ASN.1 encoding of integer values ......................................................................

pySim.euicc: Fix ASN.1 encoding of integer values

Change-Id: I26ee41705f5e95c5fa3a9997cbaebdacca3e89a7 Closes: SYS#7094 --- M pySim/euicc.py M requirements.txt M setup.py 3 files changed, 5 insertions(+), 5 deletions(-)

Approvals: osmith: Looks good to me, but someone else must approve Jenkins Builder: Verified laforge: Looks good to me, approved

diff --git a/pySim/euicc.py b/pySim/euicc.py index 6374818..069a5c7 100644 --- a/pySim/euicc.py +++ b/pySim/euicc.py @@ -177,7 +177,7 @@ class ListNotificationReq(BER_TLV_IE, tag=0xbf28, nested=[ProfileMgmtOperation]): pass class SeqNumber(BER_TLV_IE, tag=0x80): - _construct = GreedyInteger() + _construct = Asn1DerInteger() class NotificationAddress(BER_TLV_IE, tag=0x0c): _construct = Utf8Adapter(GreedyBytes) class Iccid(BER_TLV_IE, tag=0x5a): @@ -304,9 +304,9 @@ class EimIdType(BER_TLV_IE, tag=0x82): _construct = Enum(Int8ub, eimIdTypeOid=1, eimIdTypeFqdn=2, eimIdTypeProprietary=3) class CounterValue(BER_TLV_IE, tag=0x83): - _construct = GreedyInteger + _construct = Asn1DerInteger() class AssociationToken(BER_TLV_IE, tag=0x84): - _construct = GreedyInteger + _construct = Asn1DerInteger() class EimSupportedProtocol(BER_TLV_IE, tag=0x87): _construct = Enum(Int8ub, eimRetrieveHttps=0, eimRetrieveCoaps=1, eimInjectHttps=2, eimInjectCoaps=3, eimProprietary=4) diff --git a/requirements.txt b/requirements.txt index 2251a0a..58473ca 100644 --- a/requirements.txt +++ b/requirements.txt @@ -5,7 +5,7 @@ jsonpath-ng construct>=2.10.70 bidict -pyosmocom>=0.0.3 +pyosmocom>=0.0.6 pyyaml>=5.1 termcolor colorlog diff --git a/setup.py b/setup.py index aa92f17..e353f8a 100644 --- a/setup.py +++ b/setup.py @@ -24,7 +24,7 @@ "jsonpath-ng", "construct >= 2.10.70", "bidict", - "pyosmocom >= 0.0.3", + "pyosmocom >= 0.0.6", "pyyaml >= 5.1", "termcolor", "colorlog",