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

"Script execution failed" dialog when cancelling input #496

Open
ripaul opened this issue Oct 19, 2021 · 6 comments
Open

"Script execution failed" dialog when cancelling input #496

ripaul opened this issue Oct 19, 2021 · 6 comments

Comments

@ripaul
Copy link

ripaul commented Oct 19, 2021

Environment

Versions

  • Operating system: Ubuntu 20.04.1 LTS
  • Browser: Firefox 93.0
  • PassFF: 1.13
  • Host app: 1.7.3

Status line output: [12:54:54] show -> (2) gpg: decryption failed: No secret key

Observations

  • Output of pass:
  • Your preferences
    • In PassFF: Autofilling: Try to autofill page login forms.

Steps to reproduce the issue

  1. Open any page that requires a password or has a password field that will trigger passff to autofill
  2. pinentry will open, querying master password
  3. Hit Escape

Actual behaviour

After hitting Escape, a dialog opens telling you that your password could not be decrypted.

Script execution failed:
gpg: decryption failed: No secret key

image

Expected behaviour

The pinentry should close and passff should "fail silently". It may very well be intended behaviour to not autofill on a particular page. Moreover the dialog that appears cannot be closed with Escape, so you have to really click "OK" with the mouse.

This is really just a minor "issue", more like a mini enhancement.

Thanks anyways for this awesome browser plugin!

@tuxor1337
Copy link
Collaborator

Thanks for the suggestion!

I think a very quick and painless solution would be to add an option in the preferences to suppress the Script execution failed dialog whenever its spawned by the "No secret key" error message. This option would be disabled by default because users who use PassFF for the first time might run into this message due to a bad gpg setup and some kind of feedback might be helpful for them.

@madduck
Copy link
Contributor

madduck commented Oct 11, 2022

What's the status of this issue? I think it'd be great to be able to disable this dialog box.

@tuxor1337
Copy link
Collaborator

As far as I know, the proposed solution would still be valid, and nobody is working on it. So, if you or somebody else is interested, feel free to create a PR :)

@ripaul
Copy link
Author

ripaul commented Oct 11, 2022

Unfortunately I have no clue about programming firefox extensions so when I tried doing something about it I figured it'd cost me too much time, so I kinda learned to live with it. But now I just came up with a super hacky solution: In case you are using uBlock Origin (or presumably any other adblocker), you can add a filter to just block the corresponding <div id="passff_notification">...</div>.

For uBlock Origin you just add

###passff_notification

to uBlock - Dashboard > My filters.

Definitely not a clean solution and surely will have negative side effects (like e.g. the notification still popping up on pages which are not blocked by uBlock). But for the moment the best I can come up with.

@madduck
Copy link
Contributor

madduck commented Oct 11, 2022

Nice hack @ripaul, but it doesn't work for me. I will have to look into this a bit more I guess.

@tuxor1337
Copy link
Collaborator

With #544 fixed, we are now in a much better situation where this could be fixed rather easily.

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