-
Notifications
You must be signed in to change notification settings - Fork 3
[Snyk] Security upgrade urllib3 from 1.26.20 to 2.6.3 #526
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
base: main
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -7,3 +7,4 @@ arnparse==0.0.2 | |
| slack_sdk==3.22.0 | ||
| python-hcl2==4.3.1 | ||
| pre-commit>=3.5.0 | ||
| urllib3>=2.6.3 # not directly required, pinned by Snyk to avoid a vulnerability | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. urllib3 2.6.3 incompatible with pinned botocore versionHigh Severity The pinned Additional Locations (1) |
||
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.
In
smoketests/requirements.txtyou now requireurllib3>=2.6.3, butbotocore==1.31.67(and thusboto3==1.28.57) declaresurllib3<2.0, so this makes the dependency set unsatisfiable andpip install -r requirements.txt(run during the smoketests image build insmoketests/Dockerfile) will fail with a resolver error. This only occurs in environments that build the smoketests container or install these requirements, but it will block those workflows unless you also bump botocore/boto3 to versions compatible with urllib3 2.x or relax the urllib3 floor.Useful? React with 👍 / 👎.