Skip to content

Conversation

@pmenzel
Copy link
Contributor

@pmenzel Paul Menzel (pmenzel) commented Oct 13, 2026 •

AI disclaimer: This merge/pull request was generated by claude-opus-5.5 and reviewed by Paul Menzel (@pmenzel).

Commit 2c83b58 (etc/pam.d: Remove obsolete pam_ck_connector (ConsoleKit)) removed the ConsoleKit connector without adding its successor pam_systemd. So a console login on a virtual terminal is not registered with systemd-logind: it gets no session, no seat, and no device ACLs. Compositors using libseat (for example sway) therefore cannot open the DRM and input devices, when started from the console.

Add pam_systemd as optional session module after pam_loginuid, like already done in etc/pam.d/sshd, etc/pam.d/system-auth, and etc/pam.d/lightdm.

Tested on maleficent with:

env XDG_RUNTIME_DIR=/tmp/cosmic-test-$USER XDG_CURRENT_DESKTOP=COSMIC XDG_SESSION_TYPE=wayland dbus-run-session -- cosmic-session

loginctl session-status on a VT login shows Seat: seat0.

Commit 2c83b58 (etc/pam.d: Remove obsolete pam_ck_connector
(ConsoleKit)) removed the ConsoleKit connector, but did not add its
successor pam_systemd. So a console login on a virtual terminal is not
registered with systemd-logind: it gets no session, no seat, and no
device ACLs. Compositors using libseat (for example sway) therefore
cannot open the DRM and input devices, when started from the console.

Add pam_systemd as optional session module after pam_loginuid, like
already done in `etc/pam.d/sshd`, `etc/pam.d/system-auth`, and
`etc/pam.d/lightdm`.

Tested on *maleficent* with:

    env XDG_RUNTIME_DIR=/tmp/cosmic-test-$USER XDG_CURRENT_DESKTOP=COSMIC XDG_SESSION_TYPE=wayland dbus-run-session -- cosmic-session

`loginctl session-status` on a VT login shows `Seat: seat0`.

Assisted-by: claude-opus-5.5 (Claude Code)
@pmenzel Paul Menzel (pmenzel) merged commit 8e8f017 into master Oct 14, 2026
@pmenzel Paul Menzel (pmenzel) deleted the pam-login-add-pam_systemd branch October 14, 2026 05:46
Sign in to join this conversation on GitHub.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants