keepassxreboot / keepassxc

KeePassXC is a cross-platform community-driven port of the Windows application “Keepass Password Safe”.
https://keepassxc.org/
Other
21.34k stars 1.47k forks source link

"hide expired entries from auto-type" not honored with "search all open databases" #9930

Open aetonsi opened 1 year ago

aetonsi commented 1 year ago

Overview

hi, the bug i'm having is very simple: i have "hide expired entries from autotype" checked, and it works, but if i also select "search all open databases" in the autotype window, it also shows expired entries. i work with multiple dbs so "search all open databases" is used pretty often, and that makes the "hide expired entries from autotype" useless.

Steps to Reproduce

  1. have "hide expired entries from autotype" on
  2. (optional, i think) have "always ask before performing auto-type" on
  3. perform autotype
  4. select "search all open databases"

Expected Behavior

"hide expired entries from autotype" should always be honored. if i want to use an expired entry, i am doing something wrong, and/or i wouldn't have checked "hide expired entries from autotype"

Actual Behavior

expired entries are shown

Context

KeePassXC - Version 2.7.6
Revision: dd21def

Qt 5.15.10
Debugging mode is disabled.

Operating system: Windows 11 Version 2009
CPU architecture: x86_64
Kernel: winnt 10.0.22635

Enabled extensions:
- Auto-Type
- Browser Integration
- SSH Agent
- KeeShare
- YubiKey
- Quick Unlock

Cryptographic libraries:
- Botan 2.19.3
droidmonkey commented 1 year ago

Technically, that setting is only for the initial search of entries for auto-type. It's a nuance, but I can see where it is confusing.