Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

pkcs15-tool "--no-prompt" option rewording to "--use-pinpad" #944

Closed
nunojpg opened this issue Jan 22, 2017 · 9 comments
Closed

pkcs15-tool "--no-prompt" option rewording to "--use-pinpad" #944

nunojpg opened this issue Jan 22, 2017 · 9 comments

Comments

@nunojpg
Copy link
Contributor

nunojpg commented Jan 22, 2017

Would like comments to reword the "--no-prompt" to "--use-pinpad".

--no-prompt looks a terrible wording for this, and it hides the fact that a pinpad is not used by default.

Will provide a pull request according to feedback.

@nunojpg
Copy link
Contributor Author

nunojpg commented Jan 23, 2017

Under stability assurance the --no-prompt would have to be kept as an alias, possibly with a "deprecation" message for at least a full release.

@nunojpg
Copy link
Contributor Author

nunojpg commented Jan 26, 2017

I would also ask comments if pkcs15-tool (which does NOT use pinpad by default), and pkcs11-tool (which does use pinpad by default), should not be aligned...

@martinpaljak
Copy link
Member

martinpaljak commented Jan 26, 2017 via email

@frankmorgner
Copy link
Member

pkcs15-tool should only use the PIN pad if requested by the user. Look at it as low level tool.

@dengert
Copy link
Member

dengert commented Jan 28, 2017

Is there any sc_pkcs15_* routine to query if the reader is a pin pad reader?
That might be the missing piece, the pkcs15-tool can not tell if its a pin pad reader.
In that case the prompt would be "Use PIN PAD"

I understand that there are some pin pad readers that will not pass from the host a verify with a pin to the card, where others will. But I would expect if the card was inserted into a pin pad reader, the user, or the employer policy expects the user to use the pin pad.

@frankmorgner
Copy link
Member

The reader handle has a flag about the pin pad capability. However, I still don't want to let pkcs15-tool apply any magic.

@frankmorgner
Copy link
Member

I understand that there are some pin pad readers that will not pass from the host a verify with a pin to the card, where others will. But I would expect if the card was inserted into a pin pad reader, the user, or the employer policy expects the user to use the pin pad.

Indeed, both variants of readers exist. When calling pkcs15-tool the user should decide whether he wants to enter the PIN in software or on the PIN pad. That's why I'm opting for using the command line prompt as default.

@nunojpg
Copy link
Contributor Author

nunojpg commented Jan 30, 2017

@frankmorgner I understand that you don't want to align the pkcs11-tool and pkcs15-tool regarding pinpad use by default.

Regarding the wording, could you confirm if you agree it is a good idea to change it to --use-pinpad? I mean, can you make a final decision about this and close the issue?

If you agree on it I will pull request the change.

@frankmorgner
Copy link
Member

Sure, good idea. Above, I gave my 👍 long ago...

nunojpg added a commit to nunojpg/OpenSC that referenced this issue Jan 31, 2017
…enSC#944)

Wording was confused for a novice user. Old option is mantained as an alias,
but will print to stderr a deprecation warning.

Deprecation related code is all marked with deprecated word to easy future removal.

Signed-off-by: Nuno Goncalves <[email protected]>
nunojpg added a commit to nunojpg/OpenSC that referenced this issue Jan 31, 2017
…enSC#944)

Wording was confusing for a novice user. Old option is mantained as an alias,
but will print to stderr a deprecation warning.

Deprecation related code is all marked with deprecated word to easy future removal.

Signed-off-by: Nuno Goncalves <[email protected]>
nunojpg added a commit to nunojpg/OpenSC that referenced this issue Feb 1, 2017
…enSC#944)

Wording was confusing for a novice user. Old option is mantained as an alias,
but will print to stderr a deprecation warning.

Deprecation related code is all marked with deprecated word to easy future removal.

Signed-off-by: Nuno Goncalves <[email protected]>
nunojpg added a commit to nunojpg/OpenSC that referenced this issue Feb 2, 2017
…enSC#944)

Wording was confusing for a novice user. Old option is mantained as an alias,
but will print to stderr a deprecation warning.

Deprecation related code is all marked with deprecated word to easy future removal.

Signed-off-by: Nuno Goncalves <[email protected]>
nunojpg added a commit to nunojpg/OpenSC that referenced this issue Feb 2, 2017
…enSC#944)

Wording was confusing for a novice user. Old option is mantained as an alias,
but will print to stderr a deprecation warning.

Deprecation related code is all marked with deprecated word to easy future removal.

Signed-off-by: Nuno Goncalves <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants