-
Notifications
You must be signed in to change notification settings - Fork 294
Remove cryptography
upper version bound
#515
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
@vkomarov-r7 Refer: https://github.com/pyca/cryptography/blob/main/CHANGELOG.rst#3900---2023-01-01 |
I join @vkomarov-r7 in his request. It seems that only "cryptography==39.0.1" fixes CVE-2023-0401 and CVE-2023-0286 vulnerabilities. These vulnerabilities are marked as HIGH by various vulnerability scanners thus making usage of your library a security risk. Will really appreciate your effort on this one. |
This is a real blocker for us. Libs with upper bounds is bad practice for downstream users' applications. Please consider heavily removing this boundary and dealing with various version limitations in your code for compat issues. |
That makes sense. If there are backwards-incompatible changes that need to be made (and they can't be corrected in the code), would you consider releasing this change in a |
Hi all, we are testing the changes internally for supporting 39.0.1. I expect that we will have the version with the fix out in the next 2 weeks. |
This has been fixed in version |
Hey there, wanted to open a follow-on issue to #343. Would it be possible for you guys to remove the
<39.0.0
part in thecryptography
package? The same case as described previous issue has occurred again (via https://security.snyk.io/vuln/SNYK-PYTHON-CRYPTOGRAPHY-3315328).The fix is in version 39.0.1, but nobody that also depends on
oci
can update at this time.I believe this should be a safe change for projects to make based on cryptography's API Stability Policy, which states that:
The text was updated successfully, but these errors were encountered: