[PR #4915] [MERGED] keepassx: restore nou2f #5297

Closed
opened 2026-05-05 10:35:52 -06:00 by gitea-mirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/netblue30/firejail/pull/4915
Author: @kmk3
Created: 2/7/2022
Status: Merged
Merged: 2/9/2022
Merged by: @Fred-Barclay

Base: masterHead: keepassx-restore-nou2f


📝 Commits (1)

📊 Changes

1 file changed (+1 additions, -3 deletions)

View changed files

📝 etc/profile-a-l/keepassx.profile (+1 -3)

📄 Description

I could not find anything to confirm that keepassx supports hardware
keys. And as mentioned by @rusty-snake[1]:

The yubikey support in kpxc seems to be based on
https://github.com/kylemanna/keepassx /
https://github.com/keepassx/keepassx/pull/52
which was never merged. For me it looks like kpx never got official
support for it.

keepass seems to support hw keys (via plugin).

Also of note is the PR that added yubikey support to keepassxc:
https://github.com/keepassxreboot/keepassxc/pull/127

This partially reverts commit 09ac1a73e ("keepass*: remove nou2f",
2022-02-05) / PR #4903. See also commit 91b04172b ("keepass*: fix typo
in private-dev note", 2022-02-06).

Closes #4883.

[1] https://github.com/netblue30/firejail/issues/4883#issuecomment-1031172309


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/netblue30/firejail/pull/4915 **Author:** [@kmk3](https://github.com/kmk3) **Created:** 2/7/2022 **Status:** ✅ Merged **Merged:** 2/9/2022 **Merged by:** [@Fred-Barclay](https://github.com/Fred-Barclay) **Base:** `master` ← **Head:** `keepassx-restore-nou2f` --- ### 📝 Commits (1) - [`f8060a0`](https://github.com/netblue30/firejail/commit/f8060a0612537d7d7af6ff0fd50782f7dcf509c7) keepassx: restore nou2f ### 📊 Changes **1 file changed** (+1 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `etc/profile-a-l/keepassx.profile` (+1 -3) </details> ### 📄 Description I could not find anything to confirm that keepassx supports hardware keys. And as mentioned by @rusty-snake[1]: > The yubikey support in kpxc seems to be based on > https://github.com/kylemanna/keepassx / > https://github.com/keepassx/keepassx/pull/52 > which was never merged. For me it looks like kpx never got official > support for it. > > keepass seems to support hw keys (via plugin). Also of note is the PR that added yubikey support to keepassxc: https://github.com/keepassxreboot/keepassxc/pull/127 This partially reverts commit 09ac1a73e ("keepass*: remove nou2f", 2022-02-05) / PR #4903. See also commit 91b04172b ("keepass*: fix typo in private-dev note", 2022-02-06). Closes #4883. [1] https://github.com/netblue30/firejail/issues/4883#issuecomment-1031172309 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
gitea-mirror 2026-05-05 10:35:52 -06:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: github-starred/firejail#5297
No description provided.