-
Notifications
You must be signed in to change notification settings - Fork 591
Add SSL validation support for certs_keys
#1260
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
The `certs_keys` config key was added recently(ish) to the Erlang SSL stack. https://www.erlang.org/doc/apps/ssl/ssl.html#t:cert_key_conf/0
Thanks! Could we perhaps have a unit test for it? |
Sure, I'll work on that tomorrow. I couldn't find any tests for the How do you suggest I handle the OTP app logic?
|
Co-authored-by: José Valim <[email protected]>
Perhaps you could use |
Funny timing, I just thought about using Cool cool, that was my plan. I'll just test I'll have this done in the morning. I'm working on a nicer Acme integration to use for NervesHub and NervesCloud, and this is part of the puzzle. |
@josevalim done! although I did need to essentially revert your commit suggestion due to it not working as expected. |
@josevalim thank you for the merge! its almost like the good ol' days when you were mentoring me with Rails contributions :D Do you have any plans for a new release? |
The
certs_keys
config key was added recently(ish) to the Erlang SSL stack.https://www.erlang.org/doc/apps/ssl/ssl.html#t:cert_key_conf/0