Skip to content

Conversation

@tot-ra
Copy link

@tot-ra tot-ra commented May 3, 2025

snyk-top-banner

Snyk has created this PR to fix 1 vulnerabilities in the pip dependencies of this project.

Snyk changed the following file(s):

  • requirements.jetson.txt
⚠️ Warning
tensorboard 2.11.2 has requirement setuptools>=41.0.0, but you have setuptools 40.5.0.
ddtrace 2.0.2 has requirement psutil~=5.8, but you have psutil 7.0.0.

Important

  • Check the changes in this PR to ensure they won't cause issues with your project.
  • Max score is 1000. Note that the real score may have changed since the PR was raised.
  • This PR was automatically created by Snyk using the credentials of a real user.
  • Some vulnerabilities couldn't be fully fixed and so Snyk will still find them when the project is tested again. This may be because the vulnerability existed within more than one direct dependency, but not all of the affected dependencies could be upgraded.

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:

🦉 Directory Traversal

The following vulnerabilities are fixed by pinning transitive dependencies:
- https://snyk.io/vuln/SNYK-PYTHON-SETUPTOOLS-9964606
@github-actions
Copy link

github-actions bot commented May 3, 2025

There's no explicit code error in the provided diff, but there are some considerations and potential issues to be aware of:

Importance File Path Line Number(s) Comment
🔴 requirements.jetson.txt - The setuptools version has been updated to avoid a vulnerability, this is critical for the security of your project.
🟠 requirements.jetson.txt - Potential performance issue: the new version of setuptools may have different performance characteristics, it's advised to check the release notes and potentially benchmark your build process.
🟡 requirements.jetson.txt - Code inconsistency: the comment for setuptools states it's not directly required but it's pinned to avoid vulnerability. It might be clearer to remove the "not directly required" part.
🟢 requirements.jetson.txt - Improvement: you could add comments to each library in your requirements.jetson.txt file explaining why each one is needed in your project.

Additionally, the warning from Snyk about potential issues with the versions of tensorboard and ddtrace should be addressed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants