Skip to content

[codex] Start Intrade websocket quotes from subscriptions#63

Merged
NewYaroslav merged 3 commits into
mainfrom
codex/intrade-quotes-only-websockets
Jul 5, 2026
Merged

[codex] Start Intrade websocket quotes from subscriptions#63
NewYaroslav merged 3 commits into
mainfrom
codex/intrade-quotes-only-websockets

Conversation

@NewYaroslav

Copy link
Copy Markdown
Owner

What Changed

  • Made Intrade BTC and FX websocket market-data sources subscription-driven, so live websocket quotes can start without waiting for account CONNECTED.
  • Added a BTC websocket subscription refcount and default /bapi host/header setup.
  • Let FX /fxconnect streams connect as soon as a public websocket subscription is added, while preserving reconnect behavior when auth host/proxy settings change.
  • Wired BTC websocket source activation through MarketDataSubscriptionManager, including rollback and shutdown cleanup.
  • Added local websocket-server regression tests for BTCUSDT and EUR/USD websocket tick subscriptions without publishing an account-connected event.

Why

Market-data-only use should not require the full trading authentication/connect lifecycle when the broker quote endpoints are public. The public subscription API now activates websocket quote sources directly after configuration.

Validation

  • cmake --build build-codex --target intrade_bar_api_response_test -j 4
  • ./build-codex/intrade_bar_api_response_test.exe --gtest_filter=IntradeBarApiResponses.BtcWebSocketSubscriptionRunsWithoutAccountConnection:IntradeBarApiResponses.FxWebSocketSubscriptionRunsWithoutAccountConnection:IntradeBarApiResponses.BtcWebSocketSubscriptionReportsStatusAndRoutesTick:IntradeBarApiResponses.FxWebSocketSubscriptionReceivesLocalTick
  • ./build-codex/intrade_bar_api_response_test.exe

Notes

@NewYaroslav NewYaroslav force-pushed the codex/intrade-bar-status-fanout branch from 59efddd to f1e8b7d Compare July 4, 2026 16:40
@NewYaroslav NewYaroslav force-pushed the codex/intrade-quotes-only-websockets branch from 563de04 to cccc532 Compare July 4, 2026 16:40
@NewYaroslav NewYaroslav force-pushed the codex/intrade-bar-status-fanout branch from f1e8b7d to 93bb31d Compare July 4, 2026 18:03
@NewYaroslav NewYaroslav force-pushed the codex/intrade-quotes-only-websockets branch from cccc532 to 138c6ae Compare July 4, 2026 18:17
@NewYaroslav NewYaroslav force-pushed the codex/intrade-bar-status-fanout branch from 93bb31d to 4e7465a Compare July 4, 2026 19:09
@NewYaroslav NewYaroslav force-pushed the codex/intrade-quotes-only-websockets branch 2 times, most recently from 24db0b0 to 1266c71 Compare July 4, 2026 22:14
@NewYaroslav NewYaroslav force-pushed the codex/intrade-bar-status-fanout branch 2 times, most recently from c2a807e to 1ba896b Compare July 5, 2026 01:17
@NewYaroslav NewYaroslav force-pushed the codex/intrade-quotes-only-websockets branch from 1266c71 to 1741afe Compare July 5, 2026 01:17
@NewYaroslav NewYaroslav force-pushed the codex/intrade-bar-status-fanout branch from 1ba896b to eb50611 Compare July 5, 2026 01:23
@NewYaroslav NewYaroslav force-pushed the codex/intrade-quotes-only-websockets branch from 1741afe to 7dccf20 Compare July 5, 2026 01:23
@NewYaroslav NewYaroslav force-pushed the codex/intrade-bar-status-fanout branch from eb50611 to bf85e61 Compare July 5, 2026 01:36
@NewYaroslav NewYaroslav force-pushed the codex/intrade-quotes-only-websockets branch from 7dccf20 to b4330b9 Compare July 5, 2026 01:36
@NewYaroslav NewYaroslav force-pushed the codex/intrade-bar-status-fanout branch from bf85e61 to 8016e40 Compare July 5, 2026 02:28
@NewYaroslav NewYaroslav changed the base branch from codex/intrade-bar-status-fanout to main July 5, 2026 02:31
@NewYaroslav NewYaroslav force-pushed the codex/intrade-quotes-only-websockets branch from b4330b9 to 3076fa0 Compare July 5, 2026 02:31
@NewYaroslav NewYaroslav marked this pull request as ready for review July 5, 2026 03:24
@NewYaroslav NewYaroslav merged commit f9ebecf into main Jul 5, 2026
1 check passed
@NewYaroslav NewYaroslav deleted the codex/intrade-quotes-only-websockets branch July 5, 2026 03:24
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.

1 participant