Make timestamp parameter t
optional
#385
Closed
PaxxPatriot
started this conversation in
Ideas
Replies: 2 comments
-
Added in e8f9bba |
Beta Was this translation helpful? Give feedback.
0 replies
-
Thanks, I tested this already and it works perfectly fine now. Thanks for the quick implementation! |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
According to the JavaScript socket.io documentation, there is an option to deactivate the addition of the cache-busting timestamp parameter: https://socket.io/docs/v4/client-options/#timestamprequests
This is also reflected in the JavaScript code here: https://github.com/socketio/socket.io/blob/main/packages/engine.io-client/lib/transports/websocket.ts#L142. It is also disabled by default.
The behaviour of the current python-engineio implementation was blocking me from using socket.io with the Skinport socket.io Server.
So my suggestion is to make it possible to disable the addition of this parameter to the initial request URL.
Beta Was this translation helpful? Give feedback.
All reactions