Skip to content

fix(intrade-bar): tolerate transient host check failures - #170

Open
LimiNode wants to merge 1 commit into
mainfrom
fix/intrade-host-health
Open

LimiNode wants to merge 1 commit into
mainfrom
fix/intrade-host-health

Conversation

@LimiNode

Copy link
Copy Markdown
Owner

Summary

  • use GET for the current broker host health check because some proxy/front-end paths reject HEAD-like requests
  • require two consecutive failed connected host checks before disconnecting an account
  • add diagnostics and a local regression test that distinguishes GET from HEAD

Validation

  • MinGW C++17 syntax check for intrade_bar_api_response_test.cpp with the changed headers
  • git diff --check

Notes

  • full local test execution was not run in this isolated worktree because its submodules were not populated; CI runs the target.

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