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

Browser access confirmation dialog pops up on tab change #4639

Closed
droidmonkey opened this issue Apr 21, 2020 · 6 comments · Fixed by #4660
Closed

Browser access confirmation dialog pops up on tab change #4639

droidmonkey opened this issue Apr 21, 2020 · 6 comments · Fixed by #4660

Comments

@droidmonkey
Copy link
Member

Overview

This only happens while currently on a website that requires confirmation of credentials. When a new database tab is selected, the browser access confirmation dialog pops up, steals focus, and regardless of whether Allow or Deny is chosen KeePassXC minimizes.

Steps to Reproduce

  1. Go to a website that requires credential access confirmation
  2. Open multiple databases
  3. Switch between database tabs

Expected Behavior

Merely switching database tabs will not re-trigger a credential access confirmation. This should only happen when the user refreshes the page, clicks "Reload" in the extension popup menu, or requests credentials from the KPXC icon on the page.

Actual Behavior

Access confirmation is triggered every time.

Context

Occurs in 2.6.0-develop

@varjolintu
Copy link
Member

Nice find!

@varjolintu
Copy link
Member

Did you have search from all databases enabled?

@droidmonkey
Copy link
Member Author

Yes I do

@varjolintu
Copy link
Member

The problem here is that the browser extension detects the database change and requests the credentials again. I'm not sure that's something we want to prevent.

For me the access confirmation dialog stayed open when database was switched.

@droidmonkey
Copy link
Member Author

If I am not interacting with the browser there is no reason for the browser service to repoll for credentials. That is a bug IMO.

The real crime in all this is that the application gets minimized when I dismiss the credential access dialog. That behavior should be removed entirely.

@varjolintu
Copy link
Member

varjolintu commented Apr 22, 2020

If I am not interacting with the browser there is no reason for the browser service to repoll for credentials. That is a bug IMO.

This would mean user will have to refresh the page every time database is switched, and the credentials are no longer retrieved automatically. The same kind of polling happens when your database is locked and you open it. (But I could try to make it so that switching doesn't trigger the retrieval, but unlock does)

The real crime in all this is that the application gets minimized when I dismiss the credential access dialog. That behavior should be removed entirely.

That's awful indeed.

EDIT: But then again, isn't this what we decided as a wanted behaviour? If the window was minimized, it's minimized again after the access dialog?

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

Successfully merging a pull request may close this issue.

2 participants