Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Free Run ID: 📒 Files selected for processing (8)
📝 WalkthroughWalkthroughThis pull request updates the default PDS URL from Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Poem
Note 🎁 Summarized by CodeRabbit FreeYour organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Pro by visiting https://app.coderabbit.ai/login. Comment |
… extractRouteError logSafe extractRouteError already logs via logSafe (which strips JWT/DPoP/Authorization material that the atproto SDK occasionally embeds in err.cause). The three routes also called `console.error(label, err)` before invoking the helper, which both duplicated the log line and bypassed the redactSecrets pass — defeating the very mitigation logSafe exists for. Replace with a single extractRouteError call carrying a route-tagged prefix (AGENTS.md §24 #9 convention) so the resulting log line is greppable. Affected: /api/groups/[groupDid]/{profile,metadata,upload-blob} Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Summary
epds1.test.certified.apptocertified.oneacross all source files, env example, and docsapp.certifiedandapp.bskycollections via callback handlerVercel env vars (
NEXT_PUBLIC_PDS_URL) already updated for both Preview and Production.Summary by CodeRabbit
New Features
Configuration
Style
Documentation