Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

KuCoin, MEXC - Orderbook WebSockets improvements #865

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

v36u
Copy link
Contributor

@v36u v36u commented Jan 22, 2025

  1. Updated rate gate settings for KuCoin from a 60s delay for every 20 requests to 1s delay for every 60 requests (in the docs, I've seen they allow 2000 reqs / 30s for public endpoints).
  2. Moved orderbook initialization before creating the WS connection, mainly so that the caller of OnGetDeltaOrderBookWebSocketAsync has to wait for how long it actually takes to subscribe
  3. Added a useful log for MEXC, when the WS connection is rejected
  4. Made it so if there is a failure in the creation of the snapshot orderbook, it does not cause an error that interrupts the whole process and makes it start over (it now just logs)

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