Skip to content

refactor(operator)!: adopt PaymentRails; delete the tuner-app secret bypass#14

Merged
drewstone merged 1 commit into
mainfrom
feat/payment-rails
Jun 11, 2026
Merged

refactor(operator)!: adopt PaymentRails; delete the tuner-app secret bypass#14
drewstone merged 1 commit into
mainfrom
feat/payment-rails

Conversation

@drewstone

Copy link
Copy Markdown
Contributor

Greenfield cleanup, no shims.

Delete the tuner-app shared-secret bypass

The x-tuner-app-secret header + tuner_app_shared_secret config let any caller holding the secret skip all billing and serve free — a leak-prone bearer secret and scope-creep from a tuner-agent effort. Billing now applies to every caller; access control is the payment rails, not a shared secret. The two if !trusted_app_call guards become plain scopes — the intricate shielded validate/ceiling/nonce/settle logic is byte-identical.

Adopt PaymentRails

Config takes payment_rails = { shielded, direct }; the operator's provider is the PaymentRouter built from the enabled set (replaces PaymentMode).

Dependency

Pinned to tangle-inference-core rev 4476f07 (PR tangle-network/tangle-inference-core#2). Repoint to branch="master" after that merges.

Tests

server_tests 26/26 green.

…bypass

Greenfield cleanup, no shims.

- **Delete the tuner-app shared-secret bypass.** The `x-tuner-app-secret`
  header + `tuner_app_shared_secret` config let any caller with the secret skip
  ALL billing and serve free — a leak-prone bearer secret and scope-creep from a
  tuner-agent effort. Billing now applies to every caller; access control is the
  payment rails, not a shared secret. (The two `if !trusted_app_call` guards
  become plain scopes; the intricate shielded validate/ceiling/nonce/settle logic
  is byte-identical.)
- **Adopt `PaymentRails`** (replaces `PaymentMode`): config takes
  `payment_rails = { shielded, direct }`; the operator's provider is the
  `PaymentRouter` built from the enabled set.

Pinned to tangle-inference-core rev 4476f07 (feat/payment-rails). server_tests
26/26 green.

@tangletools tangletools 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.

✅ Auto-approved PR — a254e094

Blanket team auto-approval is enabled for this reviewer service.
The full PR reviewer audit still runs separately and will publish findings if it detects issues.

tangletools · auto-approval · reason: blanket_auto_approve · 2026-06-11T20:59:16Z

@drewstone drewstone merged commit 3948aee into main Jun 11, 2026
1 of 5 checks passed
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