Attention is currently required from: neels.
dexter has posted comments on this change by neels. ( https://gerrit.osmocom.org/c/pysim/+/42155?usp=email )
Change subject: esim/http_json_api.py: support text/plain response Content-Type ......................................................................
Patch Set 5:
(2 comments)
Patchset:
PS5: I think this should work, but make sure that the plaintext output is not fed into the logic that checks the header.
File pySim/esim/http_json_api.py:
https://gerrit.osmocom.org/c/pysim/+/42155/comment/f46fcf16_fba6d092?usp=ema... : PS5, Line 431: if 'header' in output: I think you should draw this header check into the conditional branch because when we decode a plaintext and it has the string "header" somewhere in it, we might generate unexpected exceptions here.