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

证书签发 #7

Merged
merged 45 commits into from
Oct 25, 2023
Merged

证书签发 #7

merged 45 commits into from
Oct 25, 2023

Conversation

VAzureV
Copy link
Collaborator

@VAzureV VAzureV commented Sep 26, 2023

No description provided.

certs/ca.crt Outdated
@@ -0,0 +1,12 @@
-----BEGIN CERTIFICATE-----
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ca.crt 和ca.pem不是一样的吗?

certs/subca.crt Outdated
@@ -0,0 +1,13 @@
-----BEGIN CERTIFICATE-----
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

subca.crt和subca.pem文件一样

sm2cert.cpp Outdated
return userCer;
}

std::shared_ptr<X509> Sm2Cert::genSignCert(std::shared_ptr<X509> midCA,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

genSignCert 和 genEncryptCert差不多,可以抽象出同一个函数,根据参数判断是签名证书还是加密证书

Copy link
Member

@dongbeiouba dongbeiouba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@dongbeiouba dongbeiouba merged commit ff316aa into Tongsuo-Project:main Oct 25, 2023
3 checks passed
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