Skip to content

fix: require explicit pass-through billing access - #5789

Merged
colegottdank merged 3 commits into
mainfrom
security/block-fraud-ptb-orgs
Aug 25, 2026
Merged

fix: require explicit pass-through billing access#5789
colegottdank merged 3 commits into
mainfrom
security/block-fraud-ptb-orgs

Conversation

@colegottdank

@colegottdank colegottdank commented Aug 23, 2026

Copy link
Copy Markdown
Collaborator

What this does

  • require the existing ptb_enabled feature flag before a PTB inference request can spend Helicone provider credits
  • require the same flag before creating a cloud-credit checkout session
  • require the same flag before scheduling or initiating an automatic top-off

What this does not touch

  • BYOK requests
  • successful Stripe payment webhooks or wallet credit reconciliation
  • subscriptions or non-PTB billing
  • provider routing after PTB access is approved

Policy

  • ptb_enabled present: PTB allowed
  • ptb_enabled absent: PTB denied
  • feature lookup failure: PTB denied

Verification

  • full Worker suite: 32 files, 1,428 tests passed
  • focused PTB/BYOK suite after final refactor: 24 tests passed
  • Jawn PTB tests: 3 passed
  • Jawn TypeScript compilation: passed
  • formatting and diff checks: passed

Rollout

  • 19 approved US orgs already have ptb_enabled
  • do not deploy to EU until its legitimate PTB orgs are reviewed and allowlisted

@vercel

vercel Bot commented Aug 23, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
helicone Ready Ready Preview Aug 24, 2026 10:37pm
helicone-eu Ready Ready Preview Aug 24, 2026 10:37pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
helicone-bifrost Skipped Skipped Aug 24, 2026 10:37pm

Request Review

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Your trial has ended. Reactivate Greptile to resume code reviews.

@claude

claude Bot commented Aug 23, 2026

Copy link
Copy Markdown
Contributor

Claude encountered an error —— View job


I'll analyze this and get back to you.

@supabase

supabase Bot commented Aug 23, 2026

Copy link
Copy Markdown

This pull request has been ignored for the connected project bolqqmqbrciybnypvklh because there are no changes detected in supbase/supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.


Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

@supabase

supabase Bot commented Aug 23, 2026

Copy link
Copy Markdown

This pull request has been ignored for the connected project lmahfbbnchpworytrrqk because there are no changes detected in .supabase/supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.


Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

@vercel
vercel Bot temporarily deployed to Preview – helicone-bifrost August 24, 2026 21:39 Inactive
@colegottdank colegottdank changed the title fix: block suspended organizations from pass-through billing fix: default-deny pass-through billing access Aug 24, 2026
@vercel
vercel Bot temporarily deployed to Preview – helicone-bifrost August 24, 2026 22:34 Inactive
@colegottdank colegottdank changed the title fix: default-deny pass-through billing access fix: require explicit pass-through billing access Aug 24, 2026
@colegottdank
colegottdank merged commit 4b81bc7 into main Aug 25, 2026
11 of 12 checks passed
@colegottdank
colegottdank deleted the security/block-fraud-ptb-orgs branch August 25, 2026 20:40
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