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

SSH Agent: Entry context menu control #4290

Merged

Conversation

hifi
Copy link
Member

@hifi hifi commented Feb 4, 2020

Allow controlling entry SSH Agent integration with hotkeys (Ctrl-H and Ctrl-Shift-H) and via context menu.

Type of change

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

Description and Context

Fixes #1815.

Screenshots

image

Testing strategy

Manually on Linux.

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]
  • ✅ I have added tests to cover my changes.

@hifi hifi force-pushed the feature/sshagent-context-menu branch 2 times, most recently from 379d450 to 39a6f4e Compare March 10, 2020 05:36
@hifi
Copy link
Member Author

hifi commented Mar 10, 2020

Note to self: this does not work with live agent enable/disable.

@hifi hifi force-pushed the feature/sshagent-context-menu branch 2 times, most recently from b4dda8a to 4d32de1 Compare March 10, 2020 18:23
@hifi hifi changed the title WIP: SSH Agent: Entry context menu control SSH Agent: Entry context menu control Mar 10, 2020
@hifi hifi requested a review from a team March 10, 2020 18:24
@hifi
Copy link
Member Author

hifi commented Mar 10, 2020

This should be good now unless something comes up.

@hifi hifi force-pushed the feature/sshagent-context-menu branch from 4d32de1 to 9c11c88 Compare March 14, 2020 18:03
Copy link
Member

@phoerious phoerious left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fix the Windows build, then it's a go.

@hifi
Copy link
Member Author

hifi commented Mar 14, 2020

Looks like a CI fluke to me. It was passing before and those tests look unrelated.

@hifi hifi force-pushed the feature/sshagent-context-menu branch from 9c11c88 to c95462a Compare March 15, 2020 05:23
@hifi
Copy link
Member Author

hifi commented Mar 15, 2020

Now MacOS tests failed randomly. The changes are fine.

@droidmonkey
Copy link
Member

Im updating the mac atm

@droidmonkey droidmonkey force-pushed the feature/sshagent-context-menu branch from c95462a to 67da7ee Compare March 19, 2020 02:43
@droidmonkey droidmonkey added this to the v2.6.0 milestone Mar 19, 2020
@droidmonkey
Copy link
Member

Fixed a crash when adding an external key to an entry. I also set it up to hide the menu actions if the current entry does not have a key.

@hifi hifi force-pushed the feature/sshagent-context-menu branch from 67da7ee to 699a3f7 Compare March 21, 2020 06:00
@hifi
Copy link
Member Author

hifi commented Mar 21, 2020

Fixed up the hiding change, squashed, reordered and rebased on develop. LGTM again.

@hifi hifi force-pushed the feature/sshagent-context-menu branch from 699a3f7 to db094a0 Compare March 21, 2020 10:25
@hifi hifi force-pushed the feature/sshagent-context-menu branch from db094a0 to c69f7e7 Compare April 4, 2020 10:30
@droidmonkey droidmonkey merged commit ef668f5 into keepassxreboot:develop Apr 8, 2020
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]
@py-crash
Copy link

py-crash commented Jul 8, 2020

Thanks for this PR. I will love this feature when the new version comes down to my distro's repo.

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.

Add SSH key to agent from entry list
4 participants