Skip to content
This repository has been archived by the owner on Oct 27, 2023. It is now read-only.

Remove unnecessary write timestamp send to websocket server. #15

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

kiennt23
Copy link

Fix this issue rootpd#2.

@tzdybal
Copy link

tzdybal commented Mar 11, 2019

I confirm, that without this fix, it's not possible to use websockets (at least for more than 1s), and fix actually fixes the problem.
Please merge ASAP.

@littleskunk
Copy link

Still not merged? Can you remove the lib from your repository? It looks like an official supported lib but its broken!

@kiennt23
Copy link
Author

Still not merged? Can you remove the lib from your repository? It looks like an official supported lib but its broken!

Not sure what do you mean. How is it broken? I don't have the right to merge, so I'm just hanging the PR here.

@littleskunk
Copy link

I am not complaining about your PR. The issue here is that binance shows no reaction!

@filinvadim
Copy link

filinvadim commented Jul 7, 2020

Removed ticker here:
https://github.com/filinvadim/go-binance

Update: i have special wss client for exchange wss endpoints - https://github.com/filinvadim/stubborn. Welcome!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants