Attention is currently required from: dexter.
osmith has posted comments on this change. ( https://gerrit.osmocom.org/c/pysim/+/34155 )
Change subject: sim-reset-server: fix error printing sw_match_error ......................................................................
Patch Set 1:
(1 comment)
File contrib/sim-rest-server.py:
https://gerrit.osmocom.org/c/pysim/+/34155/comment/c6e47279_21e638f9 PS1, Line 104: failure.value If the information inside value.error is useful, I'd suggest converting it to a string instead.
Looks like str(failure.value) would fix this. As I understand it, failure.value is of type ApiError when the error mentioned in the commit message appears. And the same line already converts an ApiError to str() the same way.
Another (IMHO more elegant) way to write this would be with f-string, which implictly converts failure.value to string:
``` return str(ApiError(f"Card Communication Error: {failure.value}"), sw) ```