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

KeePassXC determines TouchID capability somewhat unpredictably #9372

Closed
ChristopherSchultz opened this issue Apr 28, 2023 · 4 comments
Closed

Comments

@ChristopherSchultz
Copy link

I have a MacBook Pro with a fingerprint reader. When keepassxc detects that TouchID is an option, it allows me to unlock the database quickly using it and it works as expected.

The issue is when I switch back and forth between lid-open-laptop-use and external KVM with the lid closed. Sometimes touchID is offered when it's not really an option (because the lid is closed).

When launching keepassxc when the laptop is connected to an external KVM and the lid is closed, keepassxc correctly detects that TouchID is not available and does not offer it as an option. If I disconnect from the KVM and open the lid, TouchID is actually available yet keepassxc does not detect it and offer it as an authentication option.

When launching keepassxc when the laptop is standalone with the lid open, keepassxc offers TouchID and no other options, which is fine. But when re-attaching to the KVM and closing the lid, keepassxc continues to offer TouchID as the primary authentication option. You have to click "cancel" and then enter the password.

I've been trying to get this down to a 100% reproducible scenario but I'm having trouble with nailing it down. Sorry :/

@droidmonkey
Copy link
Member

We did not design touchid integration to be dynamic to lid state. This is reported in another issue and is on our radar to fix.

@droidmonkey droidmonkey closed this as not planned Won't fix, can't repro, duplicate, stale Apr 28, 2023
@ChristopherSchultz
Copy link
Author

ChristopherSchultz commented Apr 28, 2023

Glad to hear it's on the radar. Would you mind posting a link to that other issue so I can track it? (I did try to search before filing this issue!)

@droidmonkey
Copy link
Member

It's your previous issue: #8105

@ChristopherSchultz
Copy link
Author

🤭

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants