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

[v17] Restore interactive PAM authentication #49517

Merged
merged 1 commit into from
Nov 27, 2024

Conversation

rosstimothy
Copy link
Contributor

Backport #49487 to branch/v17

#29279 caused PAM to
deadlock when performing interactive authentication. To restore
the previous semblance of functional PAM, this reverts waiting
for PAM to be complete if BPF is disabled. #29279 was specifically
added to prevent systemd, which may be invoked via a PAM module,
from moving the exec subprocess to a different cgroup. Since
cgroups are not used outside of Enhanced Session Recording this
is a stop-gap measure that can allow mose users of PAM to get an
immediate restoration of behavior while a more long term and sane
approach to performing PAM during the SSH handshake can be
considered, evaluated, and tested.

Closes #49028.
@rosstimothy rosstimothy added the no-changelog Indicates that a PR does not require a changelog entry label Nov 27, 2024
@rosstimothy rosstimothy added this pull request to the merge queue Nov 27, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Nov 27, 2024
@rosstimothy rosstimothy added this pull request to the merge queue Nov 27, 2024
Merged via the queue into branch/v17 with commit 09004f1 Nov 27, 2024
41 of 42 checks passed
@rosstimothy rosstimothy deleted the bot/backport-49487-branch/v17 branch November 27, 2024 21:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport no-changelog Indicates that a PR does not require a changelog entry size/sm
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants