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.

Sign up for GitHub

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

Jump to bottom

Handle screensaver case on ledger connection #2015

Closed
fedekunze opened this issue Feb 19, 2019 · 1 comment · Fixed by #2106
Closed

Handle screensaver case on ledger connection #2015

fedekunze opened this issue Feb 19, 2019 · 1 comment · Fixed by #2106
Assignees
@fedekunze
Labels
ledger 👨‍💻 Issues related to Ledger Nano S ux-improvement issues that would improve the user experience

Comments

@fedekunze
Copy link
Contributor

fedekunze commented Feb 19, 2019 •
edited
Loading

UI Version: 0.X.X

Description:

When ledger is displaying the screensaver it polls the device successfully (i.e calls get_version successfully) but it fails to call other "protected" cryptographic functions such as getting the public key or signing.

This causes to throw an error for public key Unknown error code but not for getting the version

Ref: Zondax/ledger-cosmos-js#12
@fedekunze fedekunze added ux-improvement issues that would improve the user experience ledger 👨‍💻 Issues related to Ledger Nano S labels Feb 19, 2019
@fedekunze fedekunze self-assigned this Feb 19, 2019
@fedekunze fedekunze mentioned this issue Feb 19, 2019
Closed
@jbibla
Copy link
Collaborator

jbibla commented Feb 20, 2019

Unknown error code - could use a better error message

@fedekunze fedekunze mentioned this issue Feb 26, 2019
Merged
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@fedekunze fedekunze

Labels
ledger 👨‍💻 Issues related to Ledger Nano S ux-improvement issues that would improve the user experience
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix ledger screensaver bug luniehq/lunie
2 participants
@jbibla @fedekunze