Upgrade cryptography package to fix vulnerable OpenSSL#45040
Upgrade cryptography package to fix vulnerable OpenSSL#45040gsingh935 wants to merge 1 commit intoapache:mainfrom
Conversation
fix(security): upgrade cryptography to address vulnerability-pyca/cryptography has a vulnerable OpenSSL included in cryptography wheels
|
Congratulations on your first Pull Request and welcome to the Apache Airflow community! If you have any issues or are unsure about any anything please check our Contributors' Guide (https://github.com/apache/airflow/blob/main/contributing-docs/README.rst)
|
There was a problem hiding this comment.
@potiuk nothing more to do about constraints ? That will just be fixed in next patch release ?
potiuk
left a comment
There was a problem hiding this comment.
Those are completely different requirements - they are not airflow depdendencies - if you read README those dev/requirments.txt are only used for dev tooling - why do you want to add cryptoghraphy there @gsingh935 ? This makes no sense whatsoever. Was it some AI -generated change?
|
BTW. That issue picked my interest - and I looked closely - it seems that the "constraint" cryptography is held by yandexcloud - and I created an issue for them to address it yandex-cloud/python-sdk#131 I think it's an important one and we should even consider suspending Yandexcloud if they cannot lift the limitation. But FYI @gsingh935 -> if you are not using yandexcloud, you are absolutely free to upgrade cryptography on your own. Our constraints are just constraints, not requirements, and if you think you should upgrade a dependency for whatever reason and airflow does not "hold" the dependency as a requirement - you are on your own to do it, we are not preventing it, you just need to make sure to test if it will work for you. |
|
I am closing this issue as this was really wrongly addressing the issue. |
|
@potiuk - seems like the cryptography version for yandex-cloud has already been bumped to 43.0.1 which should address the issue. |
|
I should have looked closer. Thanks for checking @potiuk |
Upgrade cryptography package to fix vulnerable OpenSSL
GHSA-h4gh-qq45-vh27