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

fix: add pcscd polkit rule for Yubikey access #500

Closed
wants to merge 1 commit into from

Conversation

rothgar
Copy link

@rothgar rothgar commented Feb 12, 2024

I'm not exactly sure if this should be in /etc/polkit-1/rules.d or /usr/share/polkit-1/rules.d

I put it in /etc/ because it affects a user and this would allow them to change/delete the rule if needed.

@rothgar rothgar requested a review from castrojo as a code owner February 12, 2024 19:33
@rothgar rothgar changed the title Add pcscd polkit rule for Yubikey access fix: add pcscd polkit rule for Yubikey access Feb 12, 2024
@bsherman
Copy link
Contributor

@rothgar thanks for the PR!

I wouldn't add this here, instead, we've been adding rules like this to our config RPMs so they can be tracked via RPM database.

Side note: the only directly added file in this repo is our systemd-generator for sulogin. Maybe I shouldn't have even added it here, but I did so as I deem it temporary due to our goal of getting it added upstream to either Fedora or systemd itself.

It looks like your solution could be a fix to this issue there: ublue-os/config#180

I want to close this PR, but would you mind submitting a PR to https://github.com/ublue-os/config ?

It's a bit more involved as it will include tweaking the RPM spec, but I'm happy to walk you through any questions on a PR there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants