Skip to content

feat: add R2 traffic logging to Workers - #55

Closed
jinningwang wants to merge 1 commit into
mainfrom
feat/traffic-logging
Closed

jinningwang wants to merge 1 commit into
mainfrom
feat/traffic-logging

Conversation

@jinningwang

Copy link
Copy Markdown
Contributor

Summary

  • Add the shared R2 traffic logger and attach it to the REST API and MCP Workers.
  • Bind both Workers to the ps-wiki R2 bucket as TRAFFIC_LOGS.
  • Preserve existing route, middleware, response, and error-handling logic.

Validation

  • npx wrangler types passed for both Worker configurations and generated TRAFFIC_LOGS: R2Bucket.
  • npm run typecheck passed.
  • npm test passed: 7 tests.
  • npx wrangler deploy --dry-run --config wrangler.toml passed.
  • npx wrangler deploy --dry-run --config wrangler-mcp.toml passed.
  • git diff --check passed.

No deployment was performed.

@leocuisine

Copy link
Copy Markdown
Contributor

Closing this PR to recreate it under the leocuisine GitHub account; the branch and commit are unchanged.

@leocuisine leocuisine closed this Sep 14, 2026
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