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 kde and gnome wallet apps auto open in pam #1058

Closed
wants to merge 1 commit into from
Closed

Fix kde and gnome wallet apps auto open in pam #1058

wants to merge 1 commit into from

Conversation

oniGino
Copy link

@oniGino oniGino commented Jul 23, 2018

This is the fix for this issue as described in #1004

This is the fix for this issue as described in #1004
@plfiorini
Copy link
Member

Can someone with Plasma confirm it's working? @Vogtinator @davidedmundson anyone?

@Vogtinator
Copy link
Contributor

Can someone with Plasma confirm it's working? @Vogtinator @davidedmundson anyone?

The PAM files in this repo only apply to Debian/??? as they include to distro-specific files. I don't use such a distro, so can't really test that. In openSUSE the pam files are completely replaced.

@Vogtinator Vogtinator removed their request for review October 1, 2019 09:12
@a17r
Copy link
Contributor

a17r commented Oct 16, 2019

I tried, but it didn't do anything for me. that's with elogind though.

@davidosomething
Copy link

davidosomething commented Dec 15, 2019

using plasma via manjaro kde, sddm 0.18.1-1
does not work for me in the following config

#%PAM-1.0

# Order based on https://github.com/sddm/sddm/issues/1004
-auth           optional        pam_gnome_keyring.so
-auth           optional        pam_kwallet5.so
auth            include         system-login

account         include         system-login
-password     optional pam_gnome_keyring.so use_authtok
password        include         system-login

-session        optional        pam_gnome_keyring.so auto_start
-session        optional        pam_kwallet5.so auto_start
session         include         system-login

of note i am running into this error as well: https://gitlab.gnome.org/GNOME/gnome-keyring/issues/28

@plfiorini
Copy link
Member

Can someone with Plasma confirm it's working? @Vogtinator @davidedmundson anyone?

The PAM files in this repo only apply to Debian/??? as they include to distro-specific files. I don't use such a distro, so can't really test that. In openSUSE the pam files are completely replaced.

Then it's better to redirect all this issues to the distros and tell that PAM configuration is provided by the distros and they have to fix it.

@evelikov
Copy link
Contributor

evelikov commented Mar 5, 2024

With the PAM files removed as of #1856, we can close this PR.

@Vogtinator I suspect you have the super powers :-P

@oniGino oniGino closed this Mar 5, 2024
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.

None yet

6 participants