-
Notifications
You must be signed in to change notification settings - Fork 1
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
Add support for GET DATA with odd instruction #62
base: main
Are you sure you want to change the base?
Conversation
I also dived a bit deeper into the |
4d78f3f
to
2e0cc5f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If the card indicates DO handling for EF.DIR, then it should support the GET DATA com-
mand for reading all DOs in the EF at once ('00CB 2F00 02 5C00 00') directly after a reset.
I think this is not true for our implementation because apdu-dispatch
requires a SELECT
before allowing the GET DATA
command. From my understanding, we would have to add support for the EF (and maybe MF?) to apdu-dispatch
. What do you think?
src/card.rs
Outdated
/// Is command chaining supported before the Card recieves the commands | ||
pub chaining_supported: bool, | ||
|
||
/// Are exterded length supported |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s/exterded/extended/
src/command/data.rs
Outdated
00 | ||
31 F5 | ||
73 C0 01 E0 | ||
00 | ||
9000 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you please add comments naming the components?
That would seem right. Do other applications also have data in the EF.DIR and ATR? I guess |
2e0cc5f
to
c558636
Compare
c558636
to
6724f31
Compare
Looking at the fido implementation it doesn't seem to require it. |
Achim Pietig (Author of the spec) has given some clarifications regarding this feature:
But as @daringer found out the EF.DIR might be relevant for interoperability with other software:
|
I'm not sure I got it right. The documentation is unclear and
gnuk
doesn't support it.Maybe it should actually be handled by the card or in APDU dispatch because if I understand correctly the information matters to be able to correctly
SELECT
the openpgp application.