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

Unneeded password decryptions with hotkeys in action menu #212

Open
L0ric0 opened this issue Sep 5, 2021 · 3 comments
Open

Unneeded password decryptions with hotkeys in action menu #212

L0ric0 opened this issue Sep 5, 2021 · 3 comments

Comments

@L0ric0
Copy link

L0ric0 commented Sep 5, 2021

when opening the action menu with Alt+a the password is decrypted and then the dialog for the next step is shown. After choosing an action the password is decrypted again. The first decryption is not needed and is annoying for everyone who has to touch their hardware token to decrypt a password.

@hcgrove
Copy link

hcgrove commented Sep 6, 2021 via email

@L0ric0
Copy link
Author

L0ric0 commented Sep 8, 2021

I'm using the AUR package rofi-pass-ydotool-git which uses the ydotool branch. The keybindings are the ones supplied by the package.

The Alt+a keybind invokes the action_menu and there the options to choose are:

  1. move password file
  2. copy password file
  3. delete password file
  4. edit password file
  5. generate new password

I'm using a hardware token that has to be touched each time the secret key is used and when pressing Alt+a to open the action_menu it prompts me to touch it. This happens before the menu is shown. Then when choosing to edit the password I'm again prompted to touch the hardware token to decrypt the password and after editing again.

On another note I'm also using the same hardware token to sign the commits to the password-store but do see no reason why a commit or similar would be made at that point.

@carnager
Copy link
Owner

Yup, this is annoying. mainMenu function should be cleaned up quite a bit, so unneeded operations are not executed.

@carnager carnager changed the title password decrypted to early Unneeded password decryptions with hotkeys in action menu Feb 13, 2024
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

3 participants