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

Locked symbol still displayed next to username fields in browser after unlocking database #1616

Open
ep0ns opened this issue Apr 27, 2022 · 13 comments

Comments

@ep0ns
Copy link

ep0ns commented Apr 27, 2022

##Current Behaviour
When opening a page that has credential fields and the database is locked, the "keepass database locked" symbol is showed next to the username field. After unlocking the database, the locked symbol is still displayed . Only after a page reload the unlocked symbol is displayed.

Screenshot of locked symbol:
image

Expected Behavior

After unlocking the database, the correct symbol is showed next to the username on the current page. No page reload is necessary.

Steps to Reproduce (for bugs)

  1. Lock database
  2. Open any page with credential fields
  3. See the locked symbol next to the username field
  4. Click on the field to unlock the database
  5. See that still the locked symbol is displayed next to the username field, but credential filling is available

Debug info

KeePassXC - 2.7.1
KeePassXC-Browser - 1.7.12
Operating system: Win
Browser: Chrome

@varjolintu varjolintu added the bug label Apr 27, 2022
@varjolintu
Copy link
Member

varjolintu commented Apr 27, 2022

Sometimes the "broadcast" messages like database-unlocked are not retrieved properly (don't know why). This is why the status doesn't change on icons. I'll need to adjust the icon state update to a more reliable one.

@OptimusPrimeme
Copy link

I encountered a similar situation, and even if I reload the page, it still show the locked icon and the database is still locked.

@varjolintu
Copy link
Member

This might have solved it #1621. Test the current develop branch if you can, thanks.

@varjolintu
Copy link
Member

Still have the issue? It should be fixed.

@aumouvantsillage
Copy link

aumouvantsillage commented Mar 16, 2023

I am experiencing the same problem as @ep0ns. I always need to reload login pages after unlocking a database.

Browser: Firefox 110
KeePassXC version: 2.7.4
Extension version: 1.8.5.1

@varjolintu
Copy link
Member

@aumouvantsillage Anything in the JavaScript consoles?

@aumouvantsillage
Copy link

@varjolintu Nothing in the JavaScript console.

Actually, I have misread the original bug report. In the steps to reproduce, it says "the locked symbol is displayed [...] but credential filling is available".

In my case:

  • If I unlock the database by clicking on the button "Reopen the database" of the browser extension, everything works correctly.
  • If I unlock the database from the KeePassXC window, the lock symbol disappears from the extension button in the toolbar, but there is still a lock symbol in the login form. Autofilling the credentials does not work.

@varjolintu
Copy link
Member

@aumouvantsillage First this sounded like this but you are already using 2.7.4. And I remember I already fixed a bug like that in the extension. Does this happen right after reconnect? Does it matter if you refresh the web page and then unlock the database from KeePassXC?

@aumouvantsillage
Copy link

Does this happen right after reconnect?

I'm not sure what "after reconnect" means in this context.
I usually keep my computer on or in suspend mode.
On wake-up, there are usually a few web sites where I need to login again.
If the database is locked, I always need to refresh the page after unlocking.

Does it matter if you refresh the web page and then unlock the database from KeePassXC?

It makes no difference for me.

@varjolintu
Copy link
Member

@aumouvantsillage Are you using Windows? It's a reported bug that suspend interferes with the connection.

@aumouvantsillage
Copy link

No. I'm using Fedora 37.

@varjolintu
Copy link
Member

You can try to kill keepassxc-proxy and make a reconnect from the extension popup next time after you wake up from suspend.

@flops
Copy link

flops commented Sep 26, 2023

KeePassXC - 2.7.6
KeePassXC-Browser - 1.8.8
Operating system: MacIntel
Browser: Chrome/Chromium 116.0.0.0

Same issue is present for me too. If there any logs i could provide or help with debug. I'll gladly do it.

@varjolintu varjolintu removed the bug label Nov 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants