Skip to content

Show user IDs separately in audit log search results#207

Merged
yummybomb merged 2 commits into
mainfrom
hypeship/show-audit-log-user-id
Jul 17, 2026
Merged

Show user IDs separately in audit log search results#207
yummybomb merged 2 commits into
mainfrom
hypeship/show-audit-log-user-id

Conversation

@yummybomb

@yummybomb yummybomb commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add a dedicated User ID column to the default audit log search table
  • keep the User column email-only and show - when an email is unavailable instead of repeating the user ID
  • cover the displayed user ID and missing-email fallback in tests

Test plan

  • make test
  • make build
  • ./bin/kernel audit-logs search --limit 3 --no-color

Note

Low Risk
CLI-only presentation change for audit log search; no API or auth behavior changes.

Overview
kernel audit-logs search table output now has a dedicated User ID column populated from entry.UserID, so identifiers are visible even when an email is present.

The User column is email-only: it shows the address when available and - when it is not, instead of reusing the user ID in that column. JSON output is unchanged.

Tests assert user_123 appears in search table output and that formatAuditLogUser returns - when email is missing.

Reviewed by Cursor Bugbot for commit 3d0e976. Bugbot is set up for automated code reviews on this repo. Configure here.

@yummybomb yummybomb changed the title Show user IDs in audit log search results Show user IDs separately in audit log search results Jul 17, 2026
@yummybomb
yummybomb marked this pull request as ready for review July 17, 2026 15:32
@yummybomb
yummybomb requested review from Copilot and hiroTamada and removed request for Copilot July 17, 2026 15:36
@yummybomb
yummybomb merged commit 6968da5 into main Jul 17, 2026
8 checks passed
@yummybomb
yummybomb deleted the hypeship/show-audit-log-user-id branch July 17, 2026 19:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants