Skip to content

fix connection goes silent#17

Open
mussonero wants to merge 9 commits into
dmikushin:masterfrom
mussonero:master
Open

fix connection goes silent#17
mussonero wants to merge 9 commits into
dmikushin:masterfrom
mussonero:master

Conversation

@mussonero

@mussonero mussonero commented Oct 11, 2020 •
edited
Loading

Copy link
Copy Markdown
Contributor

This is a draft for schedule connection retry otherwise quit.
Still need to be optimized

@mussonero mussonero force-pushed the master branch 8 times, most recently from 3eb939a to 3e88a86 Compare October 11, 2020 20:21
@dmikushin

dmikushin commented Oct 11, 2020

Copy link
Copy Markdown
Owner

Thanks for working on this! Anything I can help with?

@mussonero

mussonero commented Oct 11, 2020

Copy link
Copy Markdown
Contributor Author

Thanks for working on this! Anything I can help with?

You are welcome :), this issue was bothering me for a long time.
At the moment I am testing one connection with a multiply streams "like more than 20 endpoints" inside one loop, it seems to work fine if the "connection attempts exhausted" or if there is an error.
Before it was not possible, even if you unplug the network cable it will not quit LOOOL.

Please take a look, if there is something to be optimized.

Kind regards
MussoNero

@mussonero

mussonero commented Oct 12, 2020

Copy link
Copy Markdown
Contributor Author

@dmikushin there are additional fixes that need to be added, I will update the PR once I finish working on it from my side.

@mussonero mussonero force-pushed the master branch 3 times, most recently from eb706ae to 0b00db1 Compare October 13, 2020 20:06
@mussonero

mussonero commented Oct 14, 2020

Copy link
Copy Markdown
Contributor Author

@dmikushin
For the moment the patch I provided seems to work fine, please let me know if I am missing something.
here is a screenshot from my auto-bots using 45 streams where it can detect the websocket session ended or it went silents
image

@mussonero mussonero force-pushed the master branch 2 times, most recently from a41a048 to b225358 Compare October 14, 2020 18:48
mussonero and others added 3 commits October 14, 2020 20:48
@mussonero
Wip: fix connection goes silent in binance_websocket
f2bde06 
schedule connection retry if it fail, it will quit
@mussonero
Additional Fixes to binance_websocket.
4f46004 
added additional notes and fixed "IF ELSE" statement +  missing header
@mussonero
fix crashing on received signal in binance_websocket
2851b5f 
Segmentation fault can happen when we attempting to emit signals from within the static method.

In other words, from the callback, we need to invoke some non-static method of WebSocket which would emit the needed signal
@mussonero mussonero force-pushed the master branch 5 times, most recently from 9c74b2b to 86f81fe Compare October 16, 2020 08:10
@mussonero
Map and serialize every endpoint connection
3282989 
Note: a single connection can listen to a maximum of 1024 streams
mussonero and others added 2 commits October 18, 2020 10:28
@mussonero
fix missing mutex_init & mutex_destroy
4c3e88a 
fix missing  pthread_mutex_init & pthread_mutex_destroy.
@mussonero
remove unused lws_extesion & more mutex luck to event-callback
649627d 
remove unused lws_extesion & more  mutex luck to event-callback
@mussonero mussonero force-pushed the master branch 2 times, most recently from dfc9d36 to 00130a4 Compare October 18, 2020 22:04
@dmikushin

dmikushin commented May 16, 2021

Copy link
Copy Markdown
Owner

@mussonero Nice progress, should I push this into master?

@dmikushin

dmikushin commented Sep 22, 2021

Copy link
Copy Markdown
Owner

Hi @mussonero , I would like to clean up your v2 version and release it under 2.0 tag in this repository. Please let me know, if you disagree.

@mussonero

mussonero commented Sep 22, 2021

Copy link
Copy Markdown
Contributor Author

Hi, yes sure no problem

@dmikushin dmikushin force-pushed the master branch 3 times, most recently from 8b945fd to 9f7c838 Compare November 19, 2024 21:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mussonero @dmikushin