-
Notifications
You must be signed in to change notification settings - Fork 70
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
Upgrade openssl version to 1.1.1n to address CVE-2022-0778 #228
Conversation
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.
LGTM
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.
Minor nit: Might be good to turn the openssl version into a shell variable / dockerfile ARG
parameter so that we can more easily make the change without so much copy-paste (DRY).
@@ -31,9 +31,9 @@ RUN curl -sSL https://github.com/Kitware/CMake/releases/download/v3.10.0/cmake-3 | |||
# Install OpenSSL 1.1.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.
@BolongZhang-AWS Can you build each of the 3 container images locally and push to docker.pkg.github.com/awslabs/aws-iot-device-client/{ubi8|ubuntu-16-x64|amazonlinux}
so that the CI builds are run inside of the containers containing the new dependencies? I am sending you a dm with the details for how to publish.
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.
Addressed this comment with additional commits on this PR.
…bff882014bcc57fd5348f758397bc
* Upgrade openssl version to 1.1.1n to address CVE-2022-0778 * Parameterize builds with OPENSSL_VERSION * Bump sdk from 15bb0b2123b0ab0a85e866c6e8d5b2713a923370 to 3223ce81919bff882014bcc57fd5348f758397bc Co-authored-by: Marco Morais <[email protected]> # Conflicts: # .github/docker-images/amazonlinux/Dockerfile # .github/docker-images/ubi8/Dockerfile # .github/docker-images/ubuntu-16-x64/Dockerfile # CMakeLists.txt.awssdk
Motivation
Modifications
Change summary
Upgrade openssl version to 1.1.1n in the build file and dockerfile.
Update the SDK commit hash in the dockerfile.
A separate PR will be submitted to upgrade the docker images.
Revision diff summary
If there is more than one revision, please explain what has been changed since the last revision.
Testing
Build the device client with the latest sdk
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.