Skip to content

Changes

Started 17 min ago
Queued 5 min 27 sec
Build has been executing for 17 min on built-in

Summary

  1. esim/http_json_api: add alternative API interface (details)
  2. ModemATCommandLink: fix SyntaxWarning: invalid escape sequence '\+' (details)
  3. es2p.py: also allow 18 digit ICCID (details)
Commit f9d7c82b4d8713ec058a80da8abe9846754f8e04 by laforge
esim/http_json_api: add alternative API interface

unfortunately the API changes introduced in change

I277aa90fddb5171c4bf6c3436259aa371d30d092

broke the API interface of http_json_api.py. This was taken into
account and necessary to introduce add the server functionality next
to the already existing client functionality. The changes to the API
were minimal and all code locations that use http_json_api.py
were re-aligned.

Unfortunately it was not clear at this point in time that there are
out-of-tree projects that could be affected by API changes in
http_json_api.py

To mitigate the problem this patch introduces an alternative API
interface to the JsonHttpApiFunction base class. This alternative
API interface works like the old API interface when the class is
instantiated in the original way. To make use of the revised client
the API use has to pass an additional keyword argument that defines
the role.

Related: SYS#7866
Change-Id: I2a5d4b59b12e08d5eae7a1215814d3a69c8921f6
The file was modifiedpySim/esim/http_json_api.py
Commit a297cdba736e1a9e6d7b3c0b0c691efb63238fe6 by laforge
ModemATCommandLink: fix SyntaxWarning: invalid escape sequence '\+'

Change-Id: If8de5299a4dc5a8525ef6657213db95d30e3c83b
Fixes: OS#6948
The file was modifiedpySim/transport/modem_atcmd.py
Commit 07a3978748101d8043df7c97f2f77c5f0ba3be7d by laforge
es2p.py: also allow 18 digit ICCID

While at it, also use tuples (const) instead of lists (var).

Tweaked-by: nhofmeyr@sysmocom.de (docstring, tuples)
Change-Id: Iaa6e710132e3f4c6cecc5ff786922f6c0fcfb54e
The file was modifiedpySim/esim/es2p.py