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
Support to WolfSSL (Step 1) #17516
Support to WolfSSL (Step 1) #17516
Conversation
CMakeLists.txt
Outdated
# OSSL_DECODER_CTX does not exist in WolfSSL | ||
set(ENABLE_ACLK False) |
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.
This is likely fine for an initial implementation, but we ideally need to update the ACLK code to work with WolfSSL.
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 agree. We will need to modify our code, but I decide to proceed as a first step.
Summary
Fixes #6509
This PR is also fixing some issues when we compile netdata disabling
ACLK
.Test Plan
wolfssl
on your system. The default compilation of the library won't allow everything necessary to haveACLK debug messages
andcloud
, to fix this I compiled it using the following options:info
endpoint, because you can test both SSL and streaming with it.Additional Information
To support ACLK and cloud, it will be necessary to do more changes in the source code. So this is the first step to bring the library.
For users: How does this change affect me?