-
Notifications
You must be signed in to change notification settings - Fork 80
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
Handle Ctrl+C to abort "Please touch the device" prompt #108
Comments
Point is, the "Please touch the device" prompt does not even show until you successfully authenticated. At least here on Ubuntu disco. That msg is useles as is. |
+1 for this. Also happy to help implement. |
@corbolais That is not the case for me, and certainly a different issue to this anyway. |
Are you using e.g.:
|
Yes, I am:
|
I've resorted to using E.g.: Instead of Ctrl+c, I just use Esc on the gui prompt. I'm sure there's a way to get a polkit prompt in the terminal, but I already have this set up for things like |
The workaround |
That still leaves the key locked for 1m. My workaround has been to press Ctrl+C and then tap the key. |
2023, still an issue 😞 |
I only have enough C programming knowledge to be dangerous, but I took a look at what it would take to support Ctrl+C when using pam-u2f with sudo. As a proof-of-concept this change worked for me: However, in the process I discovered why that naive approach is actually a bad idea (see the linked PR for details). I have no plans to continue the change forward myself, but I wanted to write up my findings to leave breadcrumbs for the next person. |
Expected: command is aborted immediately
Actual: command hangs until you touch the device or it times-out.
This has been on my wish-list for so long, I would be really happy to see this implemented. I can try implementing it as well, if you could give me some pointers?
The text was updated successfully, but these errors were encountered: