Change in pysim[master]: sim-rest-client: Errors are plain text, not JSON

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

laforge gerrit-no-reply at lists.osmocom.org
Wed Nov 3 11:34:50 UTC 2021


laforge has uploaded this change for review. ( https://gerrit.osmocom.org/c/pysim/+/26084 )


Change subject: sim-rest-client: Errors are plain text, not JSON
......................................................................

sim-rest-client: Errors are plain text, not JSON

don't try to decode JSON where there is none.

Change-Id: Iafa5d1fc20b2b9ea8d9c828fc3c7e8490d0c3693
---
M contrib/sim-rest-client.py
1 file changed, 3 insertions(+), 0 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/pysim refs/changes/84/26084/1

diff --git a/contrib/sim-rest-client.py b/contrib/sim-rest-client.py
index 8f74adc..512765b 100755
--- a/contrib/sim-rest-client.py
+++ b/contrib/sim-rest-client.py
@@ -128,6 +128,9 @@
         req_json = {'rand': rand.hex(), 'autn': t['autn'].hex()}
         print("-> %s" % req_json)
         resp = rest_post('/slot/%u' % args.slot_nr, req_json)
+        if not resp.ok:
+            print("<- ERROR %u: %s" % (resp.status_code, resp.text))
+            break
         resp_json = resp.json()
         print("<- %s" % resp_json)
         if 'synchronisation_failure' in resp_json:

-- 
To view, visit https://gerrit.osmocom.org/c/pysim/+/26084
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: Iafa5d1fc20b2b9ea8d9c828fc3c7e8490d0c3693
Gerrit-Change-Number: 26084
Gerrit-PatchSet: 1
Gerrit-Owner: laforge <laforge at osmocom.org>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20211103/202968e5/attachment.htm>


More information about the gerrit-log mailing list