feat(ord-server): add shared command builders and fee rate validator#40
feat(ord-server): add shared command builders and fee rate validator#40Neurvinch wants to merge 3 commits intoOWASP-BLT:mainfrom
Conversation
📊 Monthly LeaderboardHi @Neurvinch! Here's how you rank for March 2026:
Scoring this month (across OWASP-BLT org): Open PRs (+1 each), Merged PRs (+10), Closed (not merged) (−2), Reviews (+5; first two per PR in-month), Comments (+2, excludes CodeRabbit). Run |
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Repository: OWASP-BLT/coderabbit/.coderabbit.yml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (3)
✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
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 |
|
👋 Hi @Neurvinch! This pull request needs a peer review before it can be merged. Please request a review from a team member who is not:
Once a valid peer review is submitted, this check will pass automatically. Thank you!
|
…roxy Replace placeholder worker with a real Cloudflare Worker that: - Routes /mainnet/* and /regtest/* as API calls, everything else as static site - Validates X-Signature-256 HMAC-SHA256 signature before forwarding - Proxies authenticated API requests to the private ord backend (ORD_BACKEND_URL) - Returns JSON errors for missing config or invalid signatures - Add ORD_BACKEND_URL to wrangler.toml [vars] Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…helpers - run_ord_command: exponential-backoff retry on transient errors (connection refused, timeout, etc.) - sanitize_error: strip RPC passwords from stderr before returning - write_temp_yaml: UUID-suffixed temp files to avoid race conditions Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- make_base_command: build ord CLI prefix for mainnet/regtest - make_wallet_args: build wallet sub-command args per network - validate_fee_rate: range check (1-10000 sat/vbyte) - Fix mainnet RPC URL default port (8332, not 18443) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Neurvinch
force-pushed
the
ord/shared-command-builders
branch
from
8470816 to
6009ef8
Compare
1 participant
Summary
make_base_command(network)— builds ord CLI prefix for mainnet/regtestmake_wallet_args(network)— builds wallet sub-command args per networkvalidate_fee_rate(fee_rate)— range check (1–10000 sat/vbyte)Changes
ord-server/ord-api.py— 3 new functions, 1 bug fixTest plan
make_base_command("mainnet")returns mainnet RPC credentialsmake_base_command("regtest")includes-rflagvalidate_fee_raterejects 0, -1, 10001, "abc"; accepts 1, 500, 10000