Skip to content

[v12] Defer setting up enhanced recording until after PAM has completed#29580

Merged
r0mant merged 1 commit intobranch/v12from
tross/backport-29279/v12
Jul 25, 2023
Merged

[v12] Defer setting up enhanced recording until after PAM has completed#29580
r0mant merged 1 commit intobranch/v12from
tross/backport-29279/v12

Conversation

@rosstimothy
Copy link
Copy Markdown
Contributor

Backport #29279 to branch/v12

…9279)

The reexec process now has a two way wait mechanism to allow the
child process to complete any setup operations that may be required
before the parent process starts enhanced recording.

The old process was:

1) Parent launches child process
2) Child process opens PAM context and blocks on the continue signal
3) Parent sets up enhanced recording
4) Parent sends the continue signal
5) Child executes command/opens shell

The new process is:

1) Parent launches child process and waits for child continue signal
2) Child process opens PAM context and then signals it has completed setup
3) Parent receives child continue signal and sets up enhanced recording
4) Parent sends the continue signal
5) Child executes command/opens shell

Closes #29030
@rosstimothy rosstimothy marked this pull request as ready for review July 25, 2023 15:46
@github-actions github-actions Bot requested review from jakule and probakowski July 25, 2023 15:47
@rosstimothy rosstimothy changed the title [v12] Defer setting up enhanced recording until after PAM has completed ( [v12] Defer setting up enhanced recording until after PAM has completed Jul 25, 2023
@r0mant r0mant enabled auto-merge July 25, 2023 16:09
@public-teleport-github-review-bot public-teleport-github-review-bot Bot removed the request for review from jakule July 25, 2023 16:10
@r0mant r0mant added this pull request to the merge queue Jul 25, 2023
Merged via the queue into branch/v12 with commit 9508e30 Jul 25, 2023
@r0mant r0mant deleted the tross/backport-29279/v12 branch July 25, 2023 16:28
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