Skip to content

Fix #72: add user info to auth log events#75

Merged
mishankov merged 1 commit intomainfrom
issue-72-auth-log-event-user-info
Mar 19, 2026
Merged

Fix #72: add user info to auth log events#75
mishankov merged 1 commit intomainfrom
issue-72-auth-log-event-user-info

Conversation

@mishankov
Copy link
Copy Markdown
Member

Closes #72

Summary

  • enrich authenticated request-wide log events in the auth middleware
  • add user.id and user.username attributes when a wide event is present
  • keep the existing userId context enrichment for regular context-aware logs
  • add a focused auth middleware test covering the new wide-event behavior

Testing

  • go test ./auth
  • task test

Code Review

  • Reviewed the branch diff against origin/main
  • No material blocker findings were identified

Notes

  • task lint currently fails due to pre-existing noctx findings in auth/handler_delete_test.go, outside this change set

@mishankov mishankov force-pushed the issue-72-auth-log-event-user-info branch from 0a8af22 to d2577ae Compare March 19, 2026 05:53
@mishankov mishankov merged commit 05a5f3f into main Mar 19, 2026
6 checks passed
@mishankov mishankov deleted the issue-72-auth-log-event-user-info branch March 19, 2026 17:58
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.

Auth middleware should add user info to log event

1 participant