fix(basket): allow self-hosted ingestion without hosted billing - #805
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
2 Skipped Deployments
|
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: ASSERTIVE Plan: Advanced Run ID: 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 |
149e1c0 to
a23574f
Compare
|
The latest updates on your projects. Learn more about Unkey Deploy
|
|
Summary
Self-hosted browser events and server
/trackrequests failed with 503 because their shared quota check required an Autumn billing key. Skip that hosted billing check whenSELFHOST=true, so both ingestion paths can use the existing direct ClickHouse delivery path.Hosted behavior is preserved: with
SELFHOSTunset or false, the existing quota enforcement and billing failure responses remain unchanged. The fix adds four production lines and a regression case to the existing suite.Validation
Companion self-hosting setup work: #804. This PR branches directly from
stagingand has no dependency on another feature branch.Summary by cubic
Self-hosted browser events and server
/trackrequests no longer return 503 when hosted billing is unavailable: withSELFHOST=true, ingestion skips the Autumn quota check and uses direct ClickHouse delivery. Hosted deployments retain the existing quota enforcement and billing error behavior, with regression coverage added for the self-hosted path.Written for commit a23574f. Summary will update on new commits.