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

No such signal BrowserService::activeDatabaseChanged() #4858

Closed
yan12125 opened this issue Jun 14, 2020 · 2 comments
Closed

No such signal BrowserService::activeDatabaseChanged() #4858

yan12125 opened this issue Jun 14, 2020 · 2 comments
Assignees
Milestone

Comments

@yan12125
Copy link
Contributor

Overview

Whenever I need to confirm access requests from the browser to KeePassXC from the following dialog

image

A warning message appears in my terminal:

QObject::connect: No such signal BrowserService::activeDatabaseChanged()
QObject::connect:  (receiver name: 'BrowserAccessControlDialog')

Seems the warning is from

connect(this, SIGNAL(activeDatabaseChanged()), &accessControlDialog, SLOT(reject()));

and is related to #4660.

Steps to Reproduce

  1. Enable browser integration and enable keepassxc-proxy from the browser. Here I use Firefox but I don't think the browser type matters.
  2. Attempt to log in to some website

Expected Behavior

It works.

Actual Behavior

It works, but with a warning message in the terminal (see the Overview section for the message).

Context

KeePassXC - Version 2.6.0-snapshot
Build Type: Snapshot
Revision: c68f487

Qt 5.15.0
Debugging mode is disabled.

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

Enabled extensions:

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

Cryptographic libraries:
libgcrypt 1.8.5

I compiled keepassxc by replacing branch=develop with branch=release/2.6.0 in the AUR script.

Operating System: Arch Linux
Desktop Env: LXQt
Windowing System: X11
Browser: Firefox 77.0.1
Browser addon: KeepassXC-Browser 1.6.3

@droidmonkey
Copy link
Member

There is a really no reason for us to care about the active database, you cannot change it when this dialog is shown since it is modal.

@yan12125
Copy link
Contributor Author

Thanks for fixing this! Closing manually for the same reason as #4835.

lerignoux pushed a commit to lerignoux/keepassxc that referenced this issue Jul 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants