Skip to content

[web] Include node name or k8s cluster for session started audit entry display#49479

Merged
stevenGravy merged 2 commits intomasterfrom
stevenGravy/auditsessionstarted
Dec 5, 2024
Merged

[web] Include node name or k8s cluster for session started audit entry display#49479
stevenGravy merged 2 commits intomasterfrom
stevenGravy/auditsessionstarted

Conversation

@stevenGravy
Copy link
Copy Markdown
Contributor

@stevenGravy stevenGravy commented Nov 26, 2024

fixes #47980

This displays the node name or Kubernetes cluster information in the start session. Otherwise the user can't tell from the list which node/k8s cluster a session was started on w/o opening the details. This also informs whether it is a SSH or k8s session.

changelog: SSH or Kubernetes information included for audit log list for start session events

before:

image

updated to:

image

@stevenGravy stevenGravy marked this pull request as ready for review November 26, 2024 20:34
@stevenGravy stevenGravy changed the title Include node name or k8s cluster for session started audit entry display [web] Include node name or k8s cluster for session started audit entry display Dec 5, 2024
@stevenGravy stevenGravy requested a review from avatus December 5, 2024 15:12
@stevenGravy stevenGravy disabled auto-merge December 5, 2024 15:59
Comment thread web/packages/teleport/src/services/audit/makeEvent.ts Outdated
@stevenGravy stevenGravy enabled auto-merge December 5, 2024 16:40
@stevenGravy stevenGravy added this pull request to the merge queue Dec 5, 2024
Merged via the queue into master with commit 5196a8a Dec 5, 2024
@stevenGravy stevenGravy deleted the stevenGravy/auditsessionstarted branch December 5, 2024 16:57
@public-teleport-github-review-bot
Copy link
Copy Markdown

@stevenGravy See the table below for backport results.

Branch Result
branch/v15 Create PR
branch/v16 Create PR
branch/v17 Create PR

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.

Include node name or k8s cluster for session started audit entry display

3 participants