Skip to content
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

Add support for use certificate chain for cmd_SignCertificate and cmd… #583

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

daipingh
Copy link
Contributor

@daipingh daipingh commented Mar 4, 2024

…_EncCertificate of SSL_CONF_cmd

为 SSL_CONF_cmd 的 cmd_SignCertificate 和 cmd_EncCertificate 增加 加载证书链支持。

此pr仅供参考
如果主线希望引入此特征支持,考虑新增接口:
SSL_CTX_use_sign_certificate_chain_file
SSL_CTX_use_enc_certificate_chain_file
SSL_use_sign_certificate_chain_file
SSL_use_enc_certificate_chain_file
或类似接口(对标 SSL_CTX_use_certificate_chain_fileSSL_use_certificate_chain_file)。
本提交直接将 use_certificate_chain_file 放到 ssl_conf.c 中可能不是一个好的选择。

Checklist
  • https://yuque.com/tsdoc 增加或更新了必要的文档
  • 增加或更新了必要的测试用例
  • 对于重要修改,更新了CHANGES文件
  • 当前修改存在对已有API参数或返回值的改变
  • 当前修改存在对旧版本功能的兼容性改变(如网络协议或密码算法)

@CLAassistant
Copy link

CLAassistant commented Aug 27, 2024

CLA assistant check
All committers have signed the CLA.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants