Skip to content

Use unlock dialog when unlocking a database from browser extension#3698

Merged
droidmonkey merged 1 commit intokeepassxreboot:developfrom
varjolintu:hotfix/browser_use_unlock_dialog
May 19, 2020
Merged

Use unlock dialog when unlocking a database from browser extension#3698
droidmonkey merged 1 commit intokeepassxreboot:developfrom
varjolintu:hotfix/browser_use_unlock_dialog

Conversation

@varjolintu
Copy link
Copy Markdown
Member

Type of change

  • ✅ New feature (non-breaking change which adds functionality)

Description and Context

Instead of rising the KeePassXC window to the front, a separate unlock dialog is used.

Testing strategy

Manually.

Checklist:

  • ✅ I have read the CONTRIBUTING document. [REQUIRED]
  • ✅ My code follows the code style of this project. [REQUIRED]
  • ✅ All new and existing tests passed. [REQUIRED]
  • ✅ I have compiled and verified my code with -DWITH_ASAN=ON. [REQUIRED]

@phoerious
Copy link
Copy Markdown
Member

This PR would break effective handling of multiple databases the same way Auto-Type does atm, right?

@varjolintu
Copy link
Copy Markdown
Member Author

varjolintu commented Oct 28, 2019

@phoerious It would, yes. But it's possible to dismiss the dialog and use the main window for opening some other database instead.

EDIT: Not a nice workaround, but..

@phoerious
Copy link
Copy Markdown
Member

phoerious commented Oct 28, 2019

I would prefer we move this to 2.6 then and fix UnlockDialog first. It's unfortunate we didn't manage to do that for 2.5.

@varjolintu
Copy link
Copy Markdown
Member Author

Sure. Let's do that.

@droidmonkey
Copy link
Copy Markdown
Member

This should wait until we reconfigure this dialog to allow selecting the database to unlock.

@varjolintu varjolintu removed this from the v2.6.0 milestone May 15, 2020
@droidmonkey
Copy link
Copy Markdown
Member

Let's include this in 2.6

@varjolintu varjolintu added this to the v2.6.0 milestone May 15, 2020
@varjolintu varjolintu force-pushed the hotfix/browser_use_unlock_dialog branch from c4583f5 to 8c04db5 Compare May 15, 2020 04:10
@phoerious
Copy link
Copy Markdown
Member

We did not fix the unlock dialogue not handling multiple databases. That would be another refractor.

@droidmonkey
Copy link
Copy Markdown
Member

Right that's a 2.7 job now

@droidmonkey droidmonkey force-pushed the hotfix/browser_use_unlock_dialog branch from 8c04db5 to 6403cdb Compare May 18, 2020 14:48
@droidmonkey droidmonkey merged commit 92a7fe3 into keepassxreboot:develop May 19, 2020
@varjolintu varjolintu deleted the hotfix/browser_use_unlock_dialog branch May 19, 2020 13:06
droidmonkey added a commit that referenced this pull request Jul 7, 2020
Added

- Custom Light and Dark themes [#4110, #4769, #4791, #4796, #4892, #4915]
- Compact mode to use classic Group and Entry line height [#4910]
- View menu to quickly switch themes, compact mode, and toggle UI elements [#4910]
- Search for groups and scope search to matched groups [#4705]
- Save Database Backup feature [#4550]
- Sort entries by "natural order" and move lines up/down [#4357]
- Option to launch KeePassXC on system startup/login [#4675]
- Caps Lock warning on password input fields [#3646]
- Add "Size" column to entry view [#4588]
- Browser-like tab experience using Ctrl+[Num] (Alt+[Num] on Linux) [#4063, #4305]
- Password Generator: Define additional characters to choose from [#3876]
- Reports: Database password health check (offline) [#3993]
- Reports: HIBP online service to check for breached passwords [#4438]
- Auto-Type: DateTime placeholders [#4409]
- Browser: Show group name in results sent to browser extension [#4111]
- Browser: Ability to define a custom browser location (macOS and Linux only) [#4148]
- Browser: Ability to change root group UUID and inline edit connection ID [#4315, #4591]
- CLI: `db-info` command [#4231]
- CLI: Use wl-clipboard if xclip is not available (Linux) [#4323]
- CLI: Incorporate xclip into snap builds [#4697]
- SSH Agent: Key file path env substitution, SSH_AUTH_SOCK override, and connection test [#3769, #3801, #4545]
- SSH Agent: Context menu actions to add/remove keys [#4290]

Changed

- Complete replacement of default database icons [#4699]
- Complete replacement of application icons [#4066, #4161, #4203, #4411]
- Complete rewrite of documentation and manpages using Asciidoctor [#4937]
- Complete refactor of config files; separate between local and roaming [#4665]
- Complete refactor of browser integration and proxy code [#4680]
- Complete refactor of hardware key integration (YubiKey and OnlyKey) [#4584, #4843]
- Significantly improve performance when saving and opening databases [#4309, #4833]
- Remove read-only detection for database files [#4508]
- Overhaul of password fields and password generator [#4367]
- Replace instances of "Master Key" with "Database Credentials" [#4929]
- Change settings checkboxes to positive phrasing for consistency [#4715]
- Improve UX of using entry actions (focus fix) [#3893]
- Set expiration time to Now when enabling entry expiration [#4406]
- Always show "New Entry" in context menu [#4617]
- Issue warning before adding large attachments [#4651]
- Improve importing OPVault [#4630]
- Improve AutoOpen capability [#3901, #4752]
- Check for updates every 7 days even while still running [#4752]
- Improve Windows installer UI/UX [#4675]
- Improve config file handling of portable distribution [#4131, #4752]
- macOS: Hide dock icon when application is hidden to tray [#4782]
- Browser: Use unlock dialog to improve UX of opening a locked database [#3698]
- Browser: Improve database and entry settings experience [#4392, #4591]
- Browser: Improve confirm access dialog [#2143, #4660]
- KeeShare: Improve monitoring file changes of shares [#4720]
- CLI: Rename `create` command to `db-create` [#4231]
- CLI: Cleanup `db-create` options (`--set-key-file` and `--set-password`) [#4313]
- CLI: Use stderr for help text and password prompts [#4086, #4623]
- FdoSecrets: Display existing secret service process [#4128]

Fixed

- Fix changing focus around the main window using tab key [#4641]
- Fix search field clearing while still using the application [#4368]
- Improve search help widget displaying on macOS and Linux [#4236]
- Return keyboard focus after editing an entry [#4287]
- Reset database path after failed "Save As" [#4526]
- Use SHA256 Digest for Windows code signing [#4129]
- Improve handling of ccache when building [#4104, #4335]
- macOS: Properly re-hide application window after browser integration and Auto-Type usage [#4909]
- Auto-Type: Fix crash when performing on new entry [#4132]
- Browser: Send legacy HTTP settings to recycle bin [#4589]
- Browser: Fix merging browser keys [#4685]
- CLI: Fix encoding when exporting database [#3921]
- SSH Agent: Improve reliability and underlying code [#3833, #4256, #4549, #4595]
- FdoSecrets: Fix crash when editing settings before service is enabled [#4332]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants