-
-
Notifications
You must be signed in to change notification settings - Fork 212
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
Unsupported framework produces unhelpful error message #237
Comments
Please explain which error message would you like to get changed (and if possible, run sample logs "next" branch) |
Ah, look at that. It's only visible in the verbose output, and not indicative of whether the card will accept the software or not. I wonder if the supported frameworks can be gleaned from the package list on the card:
In this case the error (probably) was the card rejecting a too recent library, but I don't know how specific the error code is and therefore not sure about how specific the associated message should be (Unsupported plattform version vs. Card no like package). But something about the software being actively rejected is probably a good start. |
|
I don't have any strong feelings in the matter; and since the error is proprietary I can totally agree that it's outside of the scope for this project to have a pretty textual waring for it. From a UI/UX perspective; the error message could be clearer about that the error (in this case |
That sw-s come from the device should be prior knowledge of anyone working with gp. But working on the ux to make such messages clearer and better positioned would probably not hurt. Suggestions as CLI mockups would help! |
I consider this specific issue fixed by 338e53b |
Describe the bug
When trying to load an applet built with a too recent version of GlobalPlatform,
gp
produces a less-than-stellar error message.Information about your card
https://www.cardlogix.com/product/nxp-jcop3-j2h145-java-card-145k/
Expected behavior
Unsupported Framework
Full log
Using GPv1.7
Using GPv1.5 (aka 2.2.1)
Additional context
Full APDU-dump available upon request :)
The text was updated successfully, but these errors were encountered: