Skip to content

docs(nodes): restore exchange withdrawal submission - #247

Merged
TheJudii merged 1 commit into
masterfrom
agent/fix-exchange-withdrawal-config
Jul 22, 2026
Merged

docs(nodes): restore exchange withdrawal submission#247
TheJudii merged 1 commit into
masterfrom
agent/fix-exchange-withdrawal-config

Conversation

@TheJudii

Copy link
Copy Markdown
Contributor

Summary

  • restore signed transaction submission for exchange API nodes with api-accept-transactions = true
  • distinguish the transaction-accepting exchange profile from node-template's hardened read-only API and SHiP profiles
  • remove the redundant explicit eosio::producer_plugin entry and clarify that only producer_api_plugin is temporarily enabled for snapshots
  • restrict transaction write endpoints to authenticated, allowlisted exchange systems at the reverse proxy

Root cause

PR #246 changed the shared exchange API configuration to api-accept-transactions = false while retaining the Path A instructions for submitting withdrawals through push_transaction / send_transaction. TelosZero rejects those read/write API calls when transaction acceptance is disabled.

Impact

Path A, and Path B when built from the shared configuration, can once again relay valid signed withdrawal transactions. The documentation now clearly separates this private exchange role from public read-only API profiles.

Validation

  • git diff --check
  • full Docusaurus production build with yarn build

The build succeeds with the repository's existing unrelated broken-link and broken-anchor warnings.

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying telos-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 8771362
Status: ✅  Deploy successful!
Preview URL: https://9a2db1ac.telos-docs.pages.dev
Branch Preview URL: https://agent-fix-exchange-withdrawa.telos-docs.pages.dev

View logs

@TheJudii
TheJudii marked this pull request as ready for review July 22, 2026 17:06
@TheJudii
TheJudii merged commit ce8c2cd into master Jul 22, 2026
1 check 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