fix: reject throttler queue overflow#877
Conversation
PR Review: PASS (NOT VERIFIED)What This DoesMakes throttler queue overflow reject callers immediately instead of leaving pending promises. Consumers benefit on any rate-limited path under queue pressure. Blast RadiusShared core throttler utility and tests; all exchange/server code using the throttler may be affected. Consumer VerificationBefore (base branch): After (PR branch): Test Results
FindingsNo blocking findings. PMXT Pipeline Check
Semver Impactpatch -- bug fix/internal compatibility change unless SDK consumers rely on newly added APIs RiskThe branch was built, core-tested, and authenticated-smoke-tested through the sidecar. Full live-exchange before/after behavior remains unverified for this daily batch; credentialed venue calls and venue API drift should be treated as residual risk unless covered by tests in this PR. |
Covers everything between 2.48.6 and HEAD plus the hosted-trading-mode work on this branch — Added (7 bullets: hosted trading mode end-to-end, Escrow namespace, hosted error hierarchy, 2 e2e drivers, 87 new dispatch + error-mapping tests, feed listing surface), Changed (9 bullets: SDK 2.18.0 version bumps, new model fields, drift parity sweep, dep refreshes), Fixed (12 bullets covering both the hosted-mode bugs caught by the live $5 buy + sell and the unrelated fixes that landed on main since 2.48.6 — #871 #870 #873 #874 #875 #876 #877 #879 #663 #872 + the three this branch adds), Docs (2 bullets).
Summary
Tests
Fixes #690