Skip to content

[v18] fix: session end event's participants#58897

Merged
tigrato merged 2 commits intobranch/v18from
bot/backport-58856-branch/v18
Sep 9, 2025
Merged

[v18] fix: session end event's participants#58897
tigrato merged 2 commits intobranch/v18from
bot/backport-58856-branch/v18

Conversation

@tigrato
Copy link
Copy Markdown
Contributor

@tigrato tigrato commented Sep 8, 2025

Backport #58856 to branch/v18

changelog: Fixed a bug that prevented root users from viewing session recordings when they were participants.

Similarly to #58853, session's end events also showed inconsistencies in
participant fiedls for different protocols.

This created an issue where root users couldn't watch session recordings
they participated depending on the protocol, while some local users
could.

Since ithe session's participants field is a repeated string, we can't apply the
same logic as in #58853, so we need to transform the username prior to
emitting the session end events. This will allow leaf users to see their
recordings.

Fixes #12324
Fixes teleport-private#166

Signed-off-by: Tiago Silva <tiago.silva@goteleport.com>
@github-actions github-actions bot added audit-log Issues related to Teleports Audit Log backport database-access Database access related issues and PRs desktop-access kubernetes-access size/sm labels Sep 8, 2025
@tigrato tigrato enabled auto-merge September 9, 2025 08:54
@tigrato tigrato added this pull request to the merge queue Sep 9, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Sep 9, 2025
@tigrato tigrato added this pull request to the merge queue Sep 9, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Sep 9, 2025
@tigrato tigrato added this pull request to the merge queue Sep 9, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Sep 9, 2025
@tigrato tigrato added this pull request to the merge queue Sep 9, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Sep 9, 2025
@tigrato tigrato added this pull request to the merge queue Sep 9, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Sep 9, 2025
@tigrato tigrato added this pull request to the merge queue Sep 9, 2025
Merged via the queue into branch/v18 with commit 472eabd Sep 9, 2025
42 checks passed
@tigrato tigrato deleted the bot/backport-58856-branch/v18 branch September 9, 2025 12:29
@doggydogworld doggydogworld mentioned this pull request Sep 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

audit-log Issues related to Teleports Audit Log backport database-access Database access related issues and PRs desktop-access kubernetes-access size/sm

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants