Attention is currently required from: fixeria.

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

Change subject: pySim.transport: Add support for generic stdout apdu tracer ......................................................................

Patch Set 1: Code-Review+2

(1 comment)

File pySim/transport/__init__.py:

https://gerrit.osmocom.org/c/pysim/+/38054/comment/f41aa4de_25dddce0?usp=ema... : PS1, Line 258: 'apdu_tracer' in kwargs

Wondering when this condition is true. […]

we have two different apdu tracers:

1) the early stdout apdu tracer that can be enabled in the transport by this new --apdu-trace command line argument, for all of the programs using pySim.transport

2) the classic one that pySim-shell can enable at the command line with 'set apdu_trace true'. This will be in the kwargs here, and it will print using the cmd2 output functions.

The latter was insufficient as it was too late for debugging APDUs generated by pySim startup (like we read ICCID, EID, detect AIDs, etc.).