Skip to content

Restore legacy default for mariadb-connector-c server cert verification - #689

Closed
Samze wants to merge 1 commit into
cloudfoundry:developfrom
Samze:restore-mariadb-connector-c-cert-verify-default
Closed

Restore legacy default for mariadb-connector-c server cert verification#689
Samze wants to merge 1 commit into
cloudfoundry:developfrom
Samze:restore-mariadb-connector-c-cert-verify-default

Conversation

@Samze

@Samze Samze commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

mysql2 never explicitly sets MYSQL_OPT_SSL_VERIFY_SERVER_CERT, so it inherits whatever default mariadb-connector-c compiles in.

Starting with connector-c 3.4.0, that default flipped to verify the server cert (https://mariadb.com/docs?q=mariadb-connector-c-3-4-0-release-notes), which breaks ccdb connections for deployments that don't configure a CA cert (brianmario/mysql2#1379).

Building with -DDEFAULT_SSL_VERIFY_SERVER_CERT=OFF restores the pre-3.4 default of not verifying, while leaving explicit TLS configuration untouched.

Suggestion from brianmario/mysql2#1379 (comment)

Thanks for contributing to the capi_release. To speed up the process of reviewing your pull request please provide us with:

  • A short explanation of the proposed change:

  • An explanation of the use cases your change solves

  • Links to any other associated PRs

  • I have viewed signed and have submitted the Contributor License Agreement

  • I have made this pull request to the develop branch

  • I have run CF Acceptance Tests on bosh lite

mysql2 never explicitly sets MYSQL_OPT_SSL_VERIFY_SERVER_CERT, so it
inherits whatever default mariadb-connector-c compiles in. Starting
with connector-c 3.4.0, that default flipped to verify the server
cert (see mariadb.com/kb/en/mariadb-connector-c-3-4-0-release-notes),
which breaks ccdb connections for deployments that don't configure a
CA cert (see brianmario/mysql2#1379). Building with
-DDEFAULT_SSL_VERIFY_SERVER_CERT=OFF restores the pre-3.4 default of
not verifying, while leaving explicit TLS configuration untouched.
@Samze
Samze marked this pull request as draft September 1, 2026 19:28
@Samze Samze closed this Sep 1, 2026
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.

1 participant