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

[v16] Restore interactive PAM authentication #49519

Merged
merged 1 commit into from
Nov 27, 2024

Conversation

rosstimothy
Copy link
Contributor

Backport #49487 to branch/v16

Changelog: Restore interactive PAM authentication functionality when use_pam_auth is applied.

#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.
Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-49519.d212ksyjt6y4yg.amplifyapp.com

@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/v16 with commit eb00c94 Nov 27, 2024
42 of 43 checks passed
@rosstimothy rosstimothy deleted the tross/backport-49487/v16 branch November 27, 2024 21:18
@doggydogworld doggydogworld mentioned this pull request Dec 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants