Skip to content

Comments

Patch WebsocketClient to work with any websockets version >= 10.0#136

Merged
dumitrugutu merged 2 commits intomasterfrom
unpin-websockets-version
Jul 3, 2025
Merged

Patch WebsocketClient to work with any websockets version >= 10.0#136
dumitrugutu merged 2 commits intomasterfrom
unpin-websockets-version

Conversation

@dumitrugutu
Copy link
Contributor

@dumitrugutu dumitrugutu commented Jun 20, 2025

In previous versions we required websockets version >= 14 which caused issues when using the package alongside other sdk which required websockets < 14.0. This PR fixes that by downgrading the required version to >= 10.0 and dynamically setting the correct ws header param extra_headers or additional_headers for the installed version.

@dumitrugutu dumitrugutu force-pushed the unpin-websockets-version branch 3 times, most recently from 2ce9909 to 2fa417f Compare June 23, 2025 08:10
@dumitrugutu dumitrugutu requested a review from TudorCRL June 23, 2025 08:11
@dumitrugutu dumitrugutu force-pushed the unpin-websockets-version branch from 2fa417f to dec377c Compare June 30, 2025 09:47
@dumitrugutu dumitrugutu merged commit 3533e27 into master Jul 3, 2025
8 checks passed
@dumitrugutu dumitrugutu deleted the unpin-websockets-version branch July 3, 2025 11:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants