-
Notifications
You must be signed in to change notification settings - Fork 205
SG-38306 Remove Python 2 - Part 10 - SSL #372
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
base: SG-38306-python2-removal-six
Are you sure you want to change the base?
SG-38306 Remove Python 2 - Part 10 - SSL #372
Conversation
22e8204
to
c96c047
Compare
It would be great to see this merged |
What is the oldest version of python that is officially supported by this API? |
The oldest supported Python version is 3.9.
We are working on it but it is not done yet. It will take us a few more month to finish and merge it. |
6492b21
to
413afc6
Compare
4961679
to
9a131fa
Compare
413afc6
to
74a0494
Compare
docs/advanced/iron_python.rst
Outdated
For now you can workaround this problem by disabling ssl certificate validation which we've | ||
encountered some intermittent issues with. Set ``NO_SSL_VALIDATION = True`` for either case. | ||
See :const:`shotgun_api3.shotgun.NO_SSL_VALIDATION` | ||
Inject you own CA cert <<TODO>> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TODO!!!
# This is the exact same block as the "except Exception" bellow. | ||
# We need to do it here because the next except will match it | ||
# otherwise and will not re-attempt. | ||
# When we drop support of Python 2 and we will probably drop the | ||
# next except, we might want to remove this except too. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is not accurate anymore after #379. So we remove the comment but keep the block.
PR orders
WIP