-
Notifications
You must be signed in to change notification settings - Fork 17
Basic certificate support #96
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
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.
Looks very reasonable. A few API names and inout questions, but certainly is a good start. I have some concerns about the RAM necessary on the server to parse the certification chain. We may need to preallocate or use some of the existing cache memory to better handle this.
Adding this here so we don't forget: Future work should be to refactor error propagation for the verify message to return wolfCrypt error code to client, vs "catch-all" cert verification error |
Adds basic support for certificate management using wolfSSL Cert Manager.
Client API has add/delete/get functionality for trusted certificate (root), and then a verify function that verifies an (ordered) certificate chain against a given trusted root.
Limitations:
Future work: