Skip to content

fix(obs): copy security_event onto event_name - #1189

Merged
gildesmarais merged 1 commit into
mainfrom
fix/obs-security-event-name
Oct 22, 2026
Merged

gildesmarais merged 1 commit into
mainfrom
fix/obs-security-event-name

Conversation

@gildesmarais

@gildesmarais gildesmarais commented Oct 22, 2026 •

Copy link
Copy Markdown
Member

Summary

  • SecurityLogger now sets event_name to the same string as security_event so our event_name queries see token_usage / rate_limit_exceeded.
  • Specs assert both keys for token usage and rate-limit exceeded.

Test plan

  • make ready in Dev Container (exit 0)

@gildesmarais
gildesmarais merged commit 0573d02 into main Oct 22, 2026
15 checks passed
@gildesmarais
gildesmarais deleted the fix/obs-security-event-name branch October 22, 2026 09:12
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.

1 participant