You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I dislike certifi because it makes managing self-signed certificates difficult. I would prefer to use the system certificate store instead.
What solution would you like?
I believe the most straightforward solution would be to use ca_certs=None if it is explicitly set. Here's what I mean: change this
# Convert all sentinel values to their actual default# values if not using an SSLContext.ifverify_certsisVERIFY_CERTS_DEFAULT:
verify_certs=Trueifssl_show_warnisSSL_SHOW_WARN_DEFAULT:
ssl_show_warn=Trueca_certs=self.default_ca_certs() ifca_certsisNoneelseca_certs
to this
```python
# Convert all sentinel values to their actual default
# values if not using an SSLContext.
if verify_certs is VERIFY_CERTS_DEFAULT:
verify_certs = True
if ssl_show_warn is SSL_SHOW_WARN_DEFAULT:
ssl_show_warn = True
ca_certs = self.default_ca_certs() if ca_certs is CA_CERTS_DEFAULT else ca_certs
What alternatives have you considered?
I can't think of any other viable solutions.
The text was updated successfully, but these errors were encountered:
Maybe we could add ca_certs as an input parameter, introduce ca_certs = AUTO and default to it?
Yes, your option is probably better, as it is more explicit, but it deviates from the pattern of automatically determining default values used for verify_certs and ssl_show_warn. On the other hand, these are boolean variables, while we are changing str | Path | certifi | system_default. Alternatively, we could define constants like 'system' and 'certifi' to make it even clearer which paths will be checked. Please select the most preferable option. Also, are there any other suggestions for the Pull Request?
Thanks @nokados. My biggest concern is backwards compatibility, we cannot change the way existing parameters and their values work in the current interface without a major version bump, and a lesser concern future adaptability so we can keep changing it. We can always add of course. Give all these options a try and keep that in mind?
Is your feature request related to a problem?
I dislike certifi because it makes managing self-signed certificates difficult. I would prefer to use the system certificate store instead.
What solution would you like?
I believe the most straightforward solution would be to use ca_certs=None if it is explicitly set. Here's what I mean: change this
to this
What alternatives have you considered?
I can't think of any other viable solutions.
The text was updated successfully, but these errors were encountered: