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
Make ssl.PROTOCOL_* an enum #65267
Comments
Small patch to make PROTOCOL_SSLv23 and friends enum members. Not sure this is useful. |
I don't really feel real strongly one way or another about this patch fwiw. Not sure it makes anything easier but I don't think it makes anything harder either. |
It may be useful in relation with bpo-20421, since we could then return one of the enum values (not PROTOCOL_SSLv23, of course). |
Ah, sure it'd probably be useful in that context. |
I like this. Possibly it should be done for all ssl APIs returning a constant. Are there others? |
There's SSLContext.verify_mode, but it simply mirrors the configuration chosen by the user. |
Anyone else has an opinion on this? |
Looks good to me. |
New changeset f776771ab0ee by Antoine Pitrou in branch 'default': |
Ok, since this is a low-risk change I've made it anyway. |
Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.
Show more details
GitHub fields:
bugs.python.org fields:
The text was updated successfully, but these errors were encountered: