-
Notifications
You must be signed in to change notification settings - Fork 294
Use ranges in setup.py
dependencies
#343
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
Comments
Hi @vkomarov-r7, the reason that the cryptography dependency is set to v3.3.2 is we have tested with, and have compliance and security approval to use v3.3.2. We are evaluating our use of configparser and may not use it in the future. |
Hey @bhagwatvyas: Thanks for your response. Please be aware that pinning versions in this manner is itself a security vulnerability. By pinning the version of cryptography to a particular version, users of this library are unable to upgrade their For example, the currently released version of If possible, please reconsider your policy on this. Here's a list of how other popular OSS packages use the |
Hi @vkomarov-r7, the latest version of the OCI Python SDK, v 2.40.1, allows a range for cryptography versions, starting from 3.2.1 up to 3.4.7. |
Looks like this issue can be closed now, all the dependencies in setup.py have version ranges. |
In the
setup.py
file, both configparser as well as cryptography are both pinned to absolute versions. This causes version conflicts when we use theoci
library in our codebase. Would it be possible to move to using a minimum range instead?e.g.:
cryptography>=3.3.2
https://github.com/oracle/oci-python-sdk/blob/master/setup.py#L34-L35
The text was updated successfully, but these errors were encountered: