Skip to content

Fixed build against OpenSSL 4 - #6449

Open
larsewi wants to merge 1 commit into
cfengine:masterfrom
larsewi:openssl4-x509-name
Open

larsewi wants to merge 1 commit into
cfengine:masterfrom
larsewi:openssl4-x509-name

Conversation

@larsewi

@larsewi larsewi commented Oct 28, 2026 •

Copy link
Copy Markdown
Contributor

Homebrew's `openssl` now installs OpenSSL 4, which broke the macOS unit tests. In OpenSSL 4, `X509_get_subject_name()` returns a const name (see openssl/openssl@b0f2107). We used to modify it in place.

Now we build a new `X509_NAME` and set it as both subject and issuer.

🤖 Generated with Claude Code

@larsewi
larsewi marked this pull request as ready for review October 28, 2026 09:20
@larsewi

larsewi commented Oct 28, 2026

Copy link
Copy Markdown
Contributor Author

@cf-bottom Jenkins please :)

@cfengine cfengine deleted a comment from cf-bottom Oct 28, 2026
@cfengine cfengine deleted a comment from cf-bottom Oct 28, 2026
@cfengine cfengine deleted a comment from cf-bottom Oct 28, 2026
X509_get_subject_name() now returns a const name. Build a new
X509_NAME and set it as both subject and issuer instead.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Signed-off-by: Lars Erik Wik <lars.erik.wik@northern.tech>
@larsewi
larsewi force-pushed the openssl4-x509-name branch from d7727ab to f0f66f4 Compare October 28, 2026 16:35
@cfengine cfengine deleted a comment from cf-bottom Oct 28, 2026
@cf-bottom

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants