-
Notifications
You must be signed in to change notification settings - Fork 15
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
Dependencies: Switch from openssl to rustls #766
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.
Awesome!
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.
I see a few references still to OPENSSL_STATIC
in the example agents:
bottlerocket-test-system/agent/resource-agent/examples/duplicator_resource_agent/Dockerfile
Line 11 in 53f319a
ENV OPENSSL_STATIC=true |
probably makes sense to remove those too?
also this phony target to sdk-openssl
doesn't seem relevant anymore?
bottlerocket-test-system/Makefile
Line 34 in 53f319a
.PHONY: build sdk-openssl example-test-agent example-test-agent-cli example-resource-agent \ |
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.
nit: could also remove this unused sdk-openssl
.PHONY
from the makefile:
bottlerocket-test-system/Makefile
Line 34 in df0c62d
.PHONY: build sdk-openssl example-test-agent example-test-agent-cli example-resource-agent \ |
Issue number:
N/A
Description of changes:
Switches from openssl-tls to rustls-tls.
Testing done:
Make images.
Tested with
cargo make test
Terms of contribution:
By submitting this pull request, I agree that this contribution is dual-licensed under the terms of both the Apache License, version 2.0, and the MIT license.