FS#34298 - [lightdm] should include system PAM configuration

Attached to Project: Community Packages
Opened by Damjan Georgievski (damjan) - Wednesday, 13 March 2013, 14:07 GMT
Last edited by Maxime Gauduin (Alucryd) - Wednesday, 13 March 2013, 19:48 GMT
Task Type Feature Request
Category Packages
Status Closed
Assigned To Maxime Gauduin (Alucryd)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

lightdm should include the system-login pam configuration (similar to kdm), so that an admin can configure things in a single place.

This is a proposal for the lightdm.pam file

#%PAM-1.0
auth include system-login
account include system-login
password include system-login
session include system-login
This task depends upon

Closed by  Maxime Gauduin (Alucryd)
Wednesday, 13 March 2013, 19:48 GMT
Reason for closing:  Fixed
Comment by Maxime Gauduin (Alucryd) - Wednesday, 13 March 2013, 15:34 GMT
I was thinking about using the same as gentoo, and this is basically what they're doing.
lightdm: http://sources.gentoo.org/cgi-bin/viewvc.cgi/gentoo-x86/x11-misc/lightdm/files/lightdm
lightdm-autologin: http://sources.gentoo.org/cgi-bin/viewvc.cgi/gentoo-x86/x11-misc/lightdm/files/lightdm-autologin

I'll try that, and include your patch for xsession too this evening.
Comment by Maxime Gauduin (Alucryd) - Wednesday, 13 March 2013, 18:35 GMT
Okay, so the gentoo pam files didn't do wonders. Their lightdm was just your suggestion plus redundant lines (already present in our system-login), and their session line was only including system-auth, which was causing my session to fail. Yours didn't work for my GNOME session either, but I've modded it a bit and now everything works (the culprit was gnome-keyring). Also, I've just copied the gdm-autologin for lightdm-autologin because it made sense to me, gotta try autologin sometimes. new package is up, please confirm that it works and I'll close the task.
Comment by Damjan Georgievski (damjan) - Wednesday, 13 March 2013, 19:12 GMT
Maybe this?

#%PAM-1.0
auth include system-login
-auth optional pam_gnome_keyring.so

account include system-login
password include system-login

session include system-login
-session optional pam_gnome_keyring.so auto_start
Comment by Maxime Gauduin (Alucryd) - Wednesday, 13 March 2013, 19:36 GMT
This is what I did to make it work already.

Loading...