-
-
Notifications
You must be signed in to change notification settings - Fork 14.2k
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
fido2luks: move to by-name & fix build #297238
Conversation
Result of 1 package built:
|
I can't test the fix for the nixosTest, but I managed to reproduce the issue on x86_64-linux by adding another USB controller with the wrong speed. |
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/prs-ready-for-review/3032/3655 |
wait the systemd fido2 support has nothing to do with fido2luks |
a3ccd1b
to
cc21e98
Compare
I couldn't get my Nitrokey 3C NFC to work with fido2luks, so maybe I should remove the fix for nitrokeys. On the other hand, fido2luks is at least able to detect fido2 tokens using the nitrokey firmware, which is a slight improvement. |
Co-authored-by: Pavol Rusnak <[email protected]>
c2bcbc1
to
fedd84d
Compare
Description of changes
fixes #288865
I don't have a device that works with fido2luks so i can't test if it works, but it builds now at least.
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 👍 reaction to pull requests you find important.