[Snyk] Security upgrade cryptography from 45.0.7 to 46.0.6#65
[Snyk] Security upgrade cryptography from 45.0.7 to 46.0.6#65madisonmay wants to merge 1 commit intomasterfrom
Conversation
The following vulnerabilities are fixed by pinning transitive dependencies: - https://snyk.io/vuln/SNYK-PYTHON-CRYPTOGRAPHY-15809188
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: beebeee25d
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| pre-commit | ||
| -r extras/yaml.txt | ||
| -r extras/msgpack.txt | ||
| cryptography>=46.0.6 # not directly required, pinned by Snyk to avoid a vulnerability |
There was a problem hiding this comment.
Avoid forcing cryptography>=46 in Python 3.7 test envs
This new unconditional requirement applies to every tox environment that installs requirements/test.txt, including the Python 3.7 jobs still configured in tox.ini and .github/workflows/python-package.yml. cryptography 46.x no longer provides Python 3.7-compatible distributions, so this bound makes dependency resolution fail before tests run in those environments. The version floor should be guarded with a Python-version marker (or equivalent split constraint) so 3.7 jobs can still install dependencies.
Useful? React with 👍 / 👎.
Snyk has created this PR to fix 1 vulnerabilities in the pip dependencies of this project.
Snyk changed the following file(s):
requirements/test.txtImportant
Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic
Learn how to fix vulnerabilities with free interactive lessons:
🦉 Learn about vulnerability in an interactive lesson of Snyk Learn.
Note
Low Risk
Low risk dependency-only change limited to the test dependency set. Main risk is potential incompatibility in test tooling that indirectly relies on
cryptography.Overview
Pins
cryptography>=46.0.6inrequirements/test.txt(even though it’s not a direct dependency) to address a reported vulnerability in the test dependency tree.Written by Cursor Bugbot for commit beebeee. This will update automatically on new commits. Configure here.