Skip to content

Expose telemetry APIs and enforce API key rate limits#230

Merged
Makisuo merged 7 commits into
mainfrom
codex/review-code-changes
Jul 18, 2026
Merged

Expose telemetry APIs and enforce API key rate limits#230
Makisuo merged 7 commits into
mainfrom
codex/review-code-changes

Conversation

@Makisuo

@Makisuo Makisuo commented Jul 18, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Expose v2 traces, logs, metrics, services, service map, and query endpoints
  • Add stage-partitioned Cloudflare rate limiting for API-key-authenticated requests
  • Document telemetry scopes, query behavior, rate-limit responses, and log identities
  • Extend API key UI and test harnesses for telemetry scopes and rate-limit scenarios
  • Improve CORS handling and accessibility labels

Testing

  • Added HTTP coverage for rate-limit responses, fail-open behavior, session-token bypasses, and scope enforcement
  • Existing v2 test harnesses updated with telemetry and limiter layers
  • Not run (not requested)

View with Codesmith Autofix with Codesmith
Need help on this PR? Tag /codesmith with what you need. Autofix is disabled.

@pullfrog

pullfrog Bot commented Jul 18, 2026

Copy link
Copy Markdown
Contributor

Your Pullfrog Router balance is exhausted.

You have a card on file but auto-reload is disabled, so runs paused once your balance went past the overdraft buffer.

Top up balance → · Enable auto-reload →

Pullfrog  | ⚠️ this action is pinned to a commit SHA, which freezes the cleanup step — switch to @v0 or keep the SHA fresh with Dependabot | Rerun failed job ➔View workflow run | via Pullfrog | Using Claude Opus𝕏

@Makisuo
Makisuo merged commit 3246375 into main Jul 18, 2026
8 checks passed
@Makisuo
Makisuo deleted the codex/review-code-changes branch July 18, 2026 22:41
Makisuo added a commit that referenced this pull request Jul 18, 2026
The kind gate added in #230 hard-cut every pre-split standard key that
was already configured for MCP. Standard keys are full-access, so allow
them again; only scope-restricted v2 keys stay rejected.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Makisuo added a commit that referenced this pull request Jul 19, 2026
DSL-compiled queries already end with FORMAT JSON (optionally followed by
profile SETTINGS). The unconditional append added in #230 produced
'... FORMAT JSON\nFORMAT JSON', a ClickHouse syntax error that failed ~95%
of alerting queries against managed Tinybird since 2026-07-18 22:00 UTC
(alert evals, error-issue scans, anomaly detection, service-map rollups).
Only append FORMAT JSON when the query doesn't carry its own FORMAT clause.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
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