-
Notifications
You must be signed in to change notification settings - Fork 2
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
chore: vulnerability fix #39
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #39 +/- ##
=======================================
Coverage 96.69% 96.69%
=======================================
Files 5 5
Lines 363 363
=======================================
Hits 351 351
Misses 12 12 ☔ View full report in Codecov by Sentry. |
@@ -1,12 +1,12 @@ | |||
[build-system] | |||
requires = [ | |||
"setuptools >= 61.0", | |||
"setuptools >= 70.0", |
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.
Enforcing higher version for users as well. This is not needed for snyk scan.
requirements.txt
Outdated
@@ -1,5 +1,5 @@ | |||
apache-airflow == 2.10.0 | |||
requests == 2.32.3 | |||
setuptools == 65.5.1 # not directly required, pinned by Snyk to avoid a vulnerability | |||
setuptools == 70.0.0 # not directly required, pinned by Snyk to avoid a vulnerability |
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.
with pyptoject.toml, requirements.txt is not needed. Still keeping it for backward compatibility of different deployment processes.
Fixes
Introduced through
[email protected]
Fixed in
[email protected]
Part of REV-1432
Type of change
Checklist: