Skip to content

Fix Intrade FX websocket refcount connect race#82

Draft
NewYaroslav wants to merge 1 commit into
mainfrom
fix/intrade-fx-websocket-refcount
Draft

Fix Intrade FX websocket refcount connect race#82
NewYaroslav wants to merge 1 commit into
mainfrom
fix/intrade-fx-websocket-refcount

Conversation

@NewYaroslav

Copy link
Copy Markdown
Owner

Summary

  • make Intrade Bar FX websocket connect requests idempotent while a connect/reconnect is already in flight
  • keep the pending-connect state aligned with explicit disconnects and websocket reconnect lifecycle
  • harden FX websocket refcount tests so duplicate physical subscriptions are caught after the first subscribe arrives

Tests

  • cmake --build build-codex-pr68-mingw --target intrade_bar_api_response_test -j2
  • ./build-codex-pr68-mingw/intrade_bar_api_response_test.exe --gtest_brief=1
  • ./build-codex-pr68-mingw/intrade_bar_api_response_test.exe --gtest_brief=1 --gtest_filter=IntradeBarApiResponses.FxWebSocket*RefCountedBySymbol --gtest_repeat=20

@NewYaroslav NewYaroslav force-pushed the fix/intrade-fx-websocket-refcount branch from 27c4b90 to 4208fe7 Compare July 13, 2026 14:03
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