Attention is currently required from: fixeria, laforge.
dexter uploaded patch set #4 to this change.
The following approvals got outdated and were removed: Code-Review+1 by laforge, Code-Review-1 by fixeria, Verified-1 by Jenkins Builder
pySim-shell: fix CardKeyProvider for chv management commands
The CardKeyProvider support for the commands enable_chv, disable_chv,
verify_chv, change_chv and unblock_chv is broken. The reason for this
is the annotation "type=is_decimal" in the argument parser. This annotation
prevents the usage of string placeholders ("PIN1", "PUK1", etc).
Let's fix this by finding a better solution. We can also replace any
missing PIN/PUK code by checking if it is supplied or not. If not,
we query the CardKeyProvider. This also makes the usage of the *_chv
commands more uniform with the verify_adm command.
Related: OS#6531
Change-Id: I565b56ac608e801c67ca53d337bdec9efa3f3817
---
M pySim-shell.py
1 file changed, 27 insertions(+), 30 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/pysim refs/changes/38/37838/4
To view, visit change 37838. To unsubscribe, or for help writing mail filters, visit settings.