Repository navigation
Remove upper bound pin for cryptography. - #40180
Conversation
This pin is dangerous because it prevents future CVE's in a critical library from being upgraded. For example, installing the latest version of beam right now forces a version of cryptography vulnerable to CVE-2026-69247.
|
Assigning reviewers: R: @claudevdm for label python. Note: If you would like to opt out of this review, comment Available commands:
The PR bot will only process comments in the main thread (not review comments). |
|
Reminder, please take a look at this pr: @claudevdm |
|
Assigning new set of reviewers because Pr has gone too long without review. If you would like to opt out of this review, comment R: @jrmccluskey for label python. Available commands:
|
jrmccluskey
left a comment
There was a problem hiding this comment.
This is probably one of the safer deps to unpin the upper bound for given that it carries few transitive dependencies. I would also suggest pushing the lower bound up further given that 39.0.0 is nearly four years old, but that can be done in a follow-up.
This pin is dangerous because it prevents future CVE's in a critical library from being upgraded. For example, installing the latest version of beam right now forces a version of cryptography vulnerable to CVE-2026-69247.