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

Include CertificateSigning as allowed KeyUsage for the self-signed cert + add settings option to regenerate own cert #36

Merged
merged 2 commits into from
Jul 11, 2024

Conversation

basyskom-jvoe
Copy link

The certificate verification in open62541 1.4 checks for a self-signed certificate by calling X509_check_issued(). This fails with X509_V_ERR_KEYUSAGE_NO_CERTSIGN if a self-signed certificate without CertificateSigning is encountered.

The certificate verification in open62541 1.4 checks for a self-signed certificate
by calling X509_check_issued(). This fails with X509_V_ERR_KEYUSAGE_NO_CERTSIGN
if a self-signed certificate without CertificateSigning is encountered.
@basyskom-jvoe basyskom-jvoe changed the title Include CertificateSigning as allowed KeyUsage for the self-signed cert WIP: Include CertificateSigning as allowed KeyUsage for the self-signed cert Jul 10, 2024
@basyskom-jvoe basyskom-jvoe changed the title WIP: Include CertificateSigning as allowed KeyUsage for the self-signed cert Include CertificateSigning as allowed KeyUsage for the self-signed cert Jul 10, 2024
@basyskom-jvoe basyskom-jvoe changed the title Include CertificateSigning as allowed KeyUsage for the self-signed cert Include CertificateSigning as allowed KeyUsage for the self-signed cert + add settings option to regenerate own cert Jul 10, 2024
src/backend.cpp Outdated Show resolved Hide resolved
The own certificate can be viewed and replaced
@basyskom-broulik
Copy link
Contributor

Ship it

@basyskom-jvoe basyskom-jvoe merged commit ac681a9 into develop Jul 11, 2024
3 checks passed
@basyskom-jvoe basyskom-jvoe deleted the bugfix/certificate_generation branch August 5, 2024 13:35
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