Skip to content

Merge upstream changes into master #1

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

Closed
wants to merge 5 commits into from
Closed

Merge upstream changes into master #1

wants to merge 5 commits into from

Conversation

s-cork
Copy link

@s-cork s-cork commented Oct 16, 2024

Upstream has added 3.12 support see changes for ws4py/client/init.py

We should probably PR the local chagnes upstream

auvipy and others added 5 commits July 10, 2024 16:01
Signed-off-by: Asif Saif Uddin <auvipy@gmail.com>
The implementation made several months ago at Lawouach@9ffee99 works properly when turning on wss:// URLs but fails when trying to turn on server cert validation.

This fixes the issues with cert validation - I made this change on my day job's internal Gitlab, and I'm making a PR back into the public repo to contribute the fix back.
* prefer newer unittest.mock when available

* Update test/test_client.py

Signed-off-by: Asif Saif Uddin <auvipy@gmail.com>

* flake8

---------

Signed-off-by: Asif Saif Uddin <auvipy@gmail.com>
Co-authored-by: Asif Saif Uddin <auvipy@gmail.com>
* Issue Lawouach#276 fixed

* Fixed flake8 warning

---------

Co-authored-by: Vasily Zakharov <v.zakharov@wwpass.com>
@s-cork s-cork closed this Oct 16, 2024
@s-cork
Copy link
Author

s-cork commented Oct 16, 2024

We don't need this - we already removed the deprecated api - ssl.wrap_socket

@s-cork s-cork deleted the dev branch October 16, 2024 05:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants