Skip to content

pam_systemd_home: Use PAM_TEXT_INFO for token prompts#40412

Merged
poettering merged 1 commit intosystemd:mainfrom
vunnyso:vs-fixPam
Jan 21, 2026
Merged

pam_systemd_home: Use PAM_TEXT_INFO for token prompts#40412
poettering merged 1 commit intosystemd:mainfrom
vunnyso:vs-fixPam

Conversation

@vunnyso
Copy link
Copy Markdown
Contributor

@vunnyso vunnyso commented Jan 21, 2026

The prompts asking the user to physically authenticate or confirm presence on a security token are informational requests for action, not error conditions.

This commit changes the message type to PAM_TEXT_INFO, which is more appropriate for guiding the user through the authentication process.

The prompts asking the user to physically authenticate
or confirm presence on a security token are informational
requests for action, not error conditions.

This commit changes the message type to PAM_TEXT_INFO,
which is more appropriate for guiding the user through
the authentication process.
@github-actions github-actions bot added the please-review PR is ready for (re-)review by a maintainer label Jan 21, 2026
@yuwata yuwata added good-to-merge/waiting-for-ci 👍 PR is good to merge, but CI hasn't passed at time of review. Please merge if you see CI has passed homed homed, homectl, pam_homed and removed please-review PR is ready for (re-)review by a maintainer labels Jan 21, 2026
@poettering
Copy link
Copy Markdown
Member

lgtm

@poettering poettering merged commit db4b6b7 into systemd:main Jan 21, 2026
54 of 57 checks passed
@github-actions github-actions bot removed the good-to-merge/waiting-for-ci 👍 PR is good to merge, but CI hasn't passed at time of review. Please merge if you see CI has passed label Jan 21, 2026
vadika added a commit to vadika/ghaf that referenced this pull request Mar 21, 2026
The pam_systemd_home PAM_TEXT_INFO patch (systemd/systemd#40412) has been
merged upstream and is now available in the locked nixpkgs revision.

This overlay was causing a massive rebuild cascade: since systemd is a
transitive dependency of virtually every package (via dbus, polkit, udev,
glib, gtk, qt, etc.), patching it forced from-source rebuilds of packages
like inkscape, openjdk, qtbase6, and hundreds of others that would
otherwise be fetched from the binary cache.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: vadik likholetov <vadikas@gmail.com>
vadika added a commit to vadika/ghaf that referenced this pull request Mar 22, 2026
The pam_systemd_home PAM_TEXT_INFO patch (systemd/systemd#40412) has been
merged upstream and is now available in the locked nixpkgs revision.

This overlay was causing a massive rebuild cascade: since systemd is a
transitive dependency of virtually every package (via dbus, polkit, udev,
glib, gtk, qt, etc.), patching it forced from-source rebuilds of packages
like inkscape, openjdk, qtbase6, and hundreds of others that would
otherwise be fetched from the binary cache.

Signed-off-by: vadik likholetov <vadikas@gmail.com>
brianmcgillion pushed a commit to tiiuae/ghaf that referenced this pull request Mar 22, 2026
The pam_systemd_home PAM_TEXT_INFO patch (systemd/systemd#40412) has been
merged upstream and is now available in the locked nixpkgs revision.

This overlay was causing a massive rebuild cascade: since systemd is a
transitive dependency of virtually every package (via dbus, polkit, udev,
glib, gtk, qt, etc.), patching it forced from-source rebuilds of packages
like inkscape, openjdk, qtbase6, and hundreds of others that would
otherwise be fetched from the binary cache.

Signed-off-by: vadik likholetov <vadikas@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

homed homed, homectl, pam_homed

Development

Successfully merging this pull request may close these issues.

3 participants