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

Alternative/fallback PAM service #296

Open
chipb opened this issue May 7, 2023 · 2 comments
Open

Alternative/fallback PAM service #296

chipb opened this issue May 7, 2023 · 2 comments

Comments

@chipb
Copy link

chipb commented May 7, 2023

There are some rare cases where the user may not have control of the system’s /etc/pam.d directory to install a swaylock service file, but they would still like to be able to use swaylock. For example, a nix user may install swaylock to their user profile without the privileges or knowledge to also update their system’s PAM configuration in order to unlock their screen.

Since I believe these cases involve building swaylock with a different installation prefix already, I’d be inclined to add a meson build option to override the PAM service name. The user can then decide which existing system PAM service matches how they want swaylock to behave and adjust their build accordingly.

As suggested mentioned on IRC by @kennylevinsen, a fallback to the login service if swaylock is not present might be better a possibility. This has may have the benefit of Just Working without another meson option to help with a rare edge case.

(Updated wording to try avoiding misrepresentation. See issue comments immediately below.)

@kennylevinsen
Copy link
Member

As suggested on IRC by @kennylevinsen, a fallback to the login service if swaylock is not present might be better.

I strictly speaking did not suggest this, but stated that we have such fallback in greetd (for historical reasons).

@chipb
Copy link
Author

chipb commented May 8, 2023

I strictly speaking did not suggest this, but stated that we have such fallback in greetd (for historical reasons).

Sorry, I didn’t intend to put words in your mouth. I just wanted to mention that a point was raised of there being possible options aside from the big hammer of direct meson config. No solutions were endorsed, hence the opened issue.

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

No branches or pull requests

2 participants