Repository navigation
fix(api): refuse live links before saving on workspaces without a public base URL - #1084
Conversation
…lic base URL POST /feeds now checks the workspace's public base URL before the insert, so a refused create leaves no row and no longer counts toward the repo live link cap. The PR comment sync skips its live link on the same check.
|
Important Review skippedAuto reviews are limited based on label configuration. 🏷️ Required labels (at least one) (2)
🚫 Excluded labels (none allowed) (1)
Please check the settings in the CodeRabbit UI or the ⚙️ Run configuration
You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
uploads-api | a450860 | Commit Preview URL Branch Preview URL |
Oct 05 2026, 12:01 PM |
On a workspace whose storage has no public base URL,
POST /feedssaved the live link and then failed with 503feed_object_not_publicwhile building the response. The saved link stayed on the Links tab, its public page failed the same way, and a repo-wide one counted toward the 50 repo live link cap.Changes
liveLinksServable(workspace)infeed-service.ts: true when the active storage lane has a public base URL. It reads the workspace record only (no storage or credential reads).POST /feedsruns that check beforecreateFeedand refuses with the same 503feed_object_not_public. No row is saved, so the cap is not used up. This also covers scopes with no files yet, which got a link before.The web Copy live link flow already reads this 503 as "not publicly served" and shows its no-retry toast, so nothing changes there.
Testing
feed_object_not_public, and the workspace has no live links after./c/links.pnpm checkpass.Closes #1082