@CryogenicPlanetchecks…chore/remove-admin-logs-endpoint → master5 files · +0 −221review: @rohanprabhureview: @sohamganatrareview: @sarthakreview: @acsrujanreview: @kaavee315review: @abir-taheerreview: @anshugarg15updated 2w agoRemoves the /api/admin/logs/{requestId}?admin_token=... endpoint from the backend-proxy Cloudflare Worker. This endpoint exposed decrypted request logs from R2 behind a query-string admin token — killing it removes that read path entirely.
Since logs was the only route under admin/, the whole wrangler/backend-proxy/src/admin/ folder (middleware, utils, README, routes/logs.ts) is deleted along with the admin handler invocation and request-log URL print in index.ts.
The encryptedLogger that writes logs to the LOGS_BUCKET R2 binding is still used elsewhere for request logging, so that binding remains in use — only the read endpoint is gone.
grep shows no remaining references to adminRouteHandler, handleLogRoutes, getURLForAdminLogs, isAdminTokenValid, or /api/admin/logs in wrangler/.tsc --noEmit in wrangler/backend-proxy; only pre-existing unrelated errors remain (none in the changed files).🤖 Generated with Claude Code
Based on git blame analysis of 5 file(s):
| Contributor | Contribution | Files |
|---|---|---|
| jkomyno | 87% | 1 |
| Zen | 7% | 1 |
| devin-ai-integration[bot] | 3% | 1 |
| sarthak | 2% | 1 |
| Himanshu Dixit | 0% | 1 |
Based on git history, jkomyno or Zen would be good reviewer(s) for this PR.
🤖 Based on git blame with recency weighting (recent edits count more).