-
Notifications
You must be signed in to change notification settings - Fork 148
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
Implement WebSocket connection backend #469
base: master
Are you sure you want to change the base?
Conversation
I just wanted to throw my support behind this PR. I’m currently using @njboohers' It would be awesome to see this merged ASAP since it fixes the issue and gets everything back on track. Big thanks to everyone involved in pushing this forward! Maybe @rossengeorgiev can help out here? |
IIRC, during my testing, the main issue I encountered was connections to some servers didn't work, and I think it might have been due to them sending messages larger than the 16kb this reads at the time. I can't do any work on this this week, but possibly next week. |
I am using your fork in my productive Environment I have seen some errors in the log when connected to ams1 CM but they resolve after restarting the application. I will provide Logs with all the errors so we can resolve the Issues. ATM I am testing the fixes from @revij and @BlankTMing (denizkoekden@bee32ca). |
Okay I use denizkoekden@bee32ca which is your (@njbooher) Fork + the changes @revij and @BlankTMing did. The errors seem to be exactly the same as without their fixes. I was just some trial and error nobrainer testing as the base functionallity is given it just takes a few more tries to get the desired result. I use the steam lib in the steamcmd-project which is basically an simple but helpful FastAPI Project to retrive informations about an given appid. Here are the last 100 lines of the log:
|
Use TCPConnection instead of WebSocket @denizkoekden |
Resolves #114