Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Remove LegacySSL option and automatically pick the appropriate config #117

Merged
merged 1 commit into from
Jul 24, 2023
Merged

Remove LegacySSL option and automatically pick the appropriate config #117

merged 1 commit into from
Jul 24, 2023

Conversation

sepalani
Copy link
Owner

This PR removes the LegacySSL option and picks automatically the appropriate SSL config.

Ready to be reviewed & tested.

@sepalani sepalani requested a review from InusualZ July 22, 2023 13:11
InusualZ
InusualZ approved these changes Jul 22, 2023
View reviewed changes
Copy link
Collaborator

@InusualZ InusualZ left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sepalani
Copy link
Owner Author

Tested with Python 2.7, 3.9, 3.11 & working on real hardware.

@sepalani sepalani merged commit 2d3b166 into dev Jul 24, 2023
13 checks passed
@sepalani sepalani deleted the ssl-wrapper branch July 24, 2023 15:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@InusualZ InusualZ InusualZ approved these changes

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
@sepalani @InusualZ