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

Allow normal password entry at the same time as face recognition #9

Closed
boltgolt opened this issue Feb 25, 2018 · 12 comments
Closed

Allow normal password entry at the same time as face recognition #9

boltgolt opened this issue Feb 25, 2018 · 12 comments
Labels
enhancement New feature or request help wanted
Milestone

Comments

@boltgolt
Copy link
Owner

As suggested by @magarto, make Howdy asynchronous so a password can be entered at the same time though the default Unix way. This has to be done with the PAM, not sure how.

@boltgolt boltgolt added the enhancement New feature or request label Feb 25, 2018
@boltgolt
Copy link
Owner Author

I've found a doc from the fingerprint-gui module that explains how they got this principle to work.

@magarto
Copy link

magarto commented Feb 26, 2018

If I unterstand well from case 4, typing password should stop face_detection module (if not other modules are working).

Case 4 (user has typed the password)
If the PAM module returns from its password prompt with a non empty password field the user has typed his password at the prompt. Then the module sends SIGUSR2 to the helper to stop it immediately and returns PAM_IGNORE. So the other PAM modules can check the given username/password.

@boltgolt
Copy link
Owner Author

That's what I tried to do in python, but I ran into 2 issues:

First of all, the call to the PAM conversation is interpreter-blocking. This means that when we promt for a password nothing else can run. Even separate threads will freeze.

Secondly, I can't get the PAM_IGNORE signal to pass the captured password to the other PAM modules. It's doing the exact same thing as fingerprint-gui (I've run though the source) but it just falls through to a second password prompt.

@RussianNeuroMancer
Copy link

@boltgolt did you tried contact linux-pam developers about this issue?

@boltgolt
Copy link
Owner Author

I haven't, and looking at the issues page of the project it doesn't seem very active. I was working on removing the dependency in it's entirety and writing the PAM part in C++. However, i'm not that comfortable in C++ so that's on hold for now.

@RussianNeuroMancer
Copy link

@dmig
Copy link
Contributor

dmig commented Nov 19, 2018

sourceforge project seems dead, but here is some activity: https://github.com/Ralnoc/pam-python

@papermache
Copy link

papermache commented Nov 18, 2019

Enabling password entry at the same time as face recognition is becoming mandatory as authentication is not a priority of security rather an issue of urgency. If I really need access I can't wait for the camera to detect me. It's a make or break feature.

@icorbrey
Copy link

@boltgolt With your hesitation to use C++ maybe we could try using a different low-end language? There are several alternatives that would perform just as fast (if not faster) than plain old C++ (specifically Rust and Go come to mind).

@Slackadays
Copy link

Any progress on this? I thought Howdy was always asynchronous like this, but if it isn't, then I don't know if it's compatible with my (or @papermache's) workflow.

@boltgolt
Copy link
Owner Author

boltgolt commented Mar 9, 2023

You could try the beta right now and see if it works as you would expect

@saidsay-so
Copy link
Collaborator

Closed by #484.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted
Projects
None yet
Development

No branches or pull requests

8 participants