Skip to content

AutoOpen only works if password is not empty #3931

@bwbroersma

Description

@bwbroersma

When using AutoOpen, a password must be present to open another database.

Expected Behavior

AutoOpen would work just with an url (the database), or url + username (the keyfile).

Current Behavior

If no password is present, AutoOpen will not work.

Possible Solution

Remove the || entry->password().isEmpty() or change it to || (entry->password().isEmpty() && entry->username().isEmpty()). The first solution would work for databases without password and keyfile, the second enabled just keyfile without password AutoOpen.

for (const auto* entry : autoopenGroup->entries()) {
if (entry->url().isEmpty() || entry->password().isEmpty()) {
continue;
}

Steps to Reproduce

  1. Add folder /AutoOpen (directly under root)
  2. Add entry with an url and username but without a password
  3. Save database
  4. Lock database
  5. Unlock database

Context

Debug Info

KeePassXC - Version 2.5.1
Revision: 0fd8836
Distribution: AppImage

Qt 5.12.3
Debugging mode is disabled.

Operating system: Debian GNU/Linux 10 (buster)
CPU architecture: x86_64
Kernel: linux 5.2.0-0.bpo.3-amd64

Enabled extensions:

  • Auto-Type
  • Browser Integration
  • SSH Agent
  • KeeShare (signed and unsigned sharing)
  • YubiKey
  • Secret Service Integration

Cryptographic libraries:
libgcrypt 1.8.1

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions