keepassxc icon indicating copy to clipboard operation
keepassxc copied to clipboard

Referenced Passwords Show "Strong" When Editing

Open WinnerWind opened this issue 3 months ago • 3 comments

Have you searched for an existing issue?

  • [x] Yes, I tried searching and reviewed the pinned issues

Brief Summary

When using a password that references another entry, the password is shown as "strong" even though the password it refers to itself is weak.

Image Image

Note that the passwords are correctly shown as weak in the database view.

Image

If i had to guess, it's because the reference is not being dereferenced / fetched before the password strength is determined, but only in the editing menu. The database view fetches the referenced password first.

Steps to Reproduce

  1. Create a weak password entry
  2. Clone that entry / Create a reference to the password in that entry
  3. See that the weak password is shown as weak, but the new cloned / referenced entry has a 'strong' password.

Expected Versus Actual Behavior

Expected: KeePassXC will show me that the referenced password is weak in the edit menu

Actual: KeePassXC insists the referenced password is strong

KeePassXC Debug Information

KeePassXC - Version 2.7.10
Revision: b342be4

Qt 5.15.18
Debugging mode is disabled.

Operating system: Arch Linux
CPU architecture: x86_64
Kernel: linux 6.17.7-arch1-1

Enabled extensions:
- Auto-Type
- Browser Integration
- Passkeys
- SSH Agent
- KeeShare
- YubiKey
- Secret Service Integration

Cryptographic libraries:
- Botan 3.9.0

Operating System

Linux

Linux Desktop Environment

KDE

Linux Windowing System

Wayland

WinnerWind avatar Nov 12 '25 16:11 WinnerWind

Good catch!

droidmonkey avatar Nov 12 '25 17:11 droidmonkey

No worries! As a programmer myself I saw this one coming from a mile away, haha. We all make mistakes

WinnerWind avatar Nov 12 '25 17:11 WinnerWind

Good day, I already have solution for the issue, however I came up to life-cycle question. Due to the temporary m_entry before it is committed to the db that is owned by the MainWindow I cannot make it work before the entry is created. I would either need clarification how to proceed or I can create a PR with current changed, and then we can discuss how to change life-cycle (maybe you already have a solution that I cannot see). Thank you and have a great day.

daxsis avatar Nov 16 '25 11:11 daxsis