Attention is currently required from: pespin.

laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/pysim/+/34946?usp=email )

Change subject: pySim-shell: Validate that argument to 'apdu' command is proper hexstr ......................................................................

Patch Set 2:

(1 comment)

File pySim/utils.py:

https://gerrit.osmocom.org/c/pysim/+/34946/comment/75ddfb44_d8428a9d PS1, Line 1476: if not all(c in string.hexdigits for c in instr):

you can probably return is_hexstr here.

nice idea, but the error strings would then be potentially confusing. Like getting "Input must be hexadecimal" when calling is_hexstr_or_decimal, which currently states "Input must be [hexa]decimal"...