Skip to content

Database secured connection#23

Open
bdalsass wants to merge 16 commits intomasterfrom
feature/add-database-certificates
Open

Database secured connection#23
bdalsass wants to merge 16 commits intomasterfrom
feature/add-database-certificates

Conversation

@bdalsass
Copy link
Copy Markdown
Collaborator

Base information

Question Answer
Related to a SourceForge thread / Another PR / Combodo ticket? NO
Type of change? Enhancement

Symptom (bug) / Objective (enhancement)

Add configuration for secured connection between database server and client

@bdalsass bdalsass changed the title Feature/add database certificates Database secured connection Apr 30, 2026
@bdalsass bdalsass requested review from Lenaick, Molkobain and Copilot May 4, 2026 05:57
@bdalsass bdalsass self-assigned this May 4, 2026
@bdalsass bdalsass added the enhancement New feature or request label May 4, 2026
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds optional TLS/SSL configuration for MariaDB/MySQL connections in the Docker iTop environment, wiring certificate mounts into the DB/client containers and documenting how to enable secure transport.

Changes:

  • Add database TLS configuration (cert paths, TLS versions/ciphers, require_secure_transport) for MySQL/MariaDB.
  • Mount DB/webserver certificate directories into services (PHP, DBs, Adminer) and add a PHP client config (client.cnf).
  • Extend documentation to describe TLS enablement and Adminer SSL usage.

Reviewed changes

Copilot reviewed 16 out of 19 changed files in this pull request and generated 18 comments.

Show a summary per file
File Description
readme.md Updates container descriptions and mentions DB TLS/cert availability.
docker-compose.yml Mounts certs into PHP/DB/Adminer and adds Adminer SSL env vars/plugins mount.
build/default_configuration/php/client.cnf Adds MySQL/MariaDB client TLS configuration.
build/default_configuration/mysql/my.cnf Adds MySQL server TLS settings and require_secure_transport.
build/default_configuration/mariadb/my.cnf Adds MariaDB server TLS settings and require_secure_transport.
build/default_configuration/certs/webserver/certificate.key Adds bundled webserver private key (sensitive).
build/default_configuration/certs/webserver/certificate.crt Adds bundled webserver certificate.
build/default_configuration/certs/database/* Adds CA/client/server certs and private keys for DB TLS.
build/default_configuration/adminer/plugins-enabled/enable-ssl;php.php Adds Adminer plugin to enable SSL connections.
addons/documentation/how_to.md Adds/updates TLS and dump import/export guidance.
addons/docker_scripts/down_and_up.sh Adds helper script to restart compose with .env.local.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread build/default_configuration/certs/database/server-key.pem
Comment thread docker-compose.yml Outdated
Comment thread docker-compose.yml Outdated
Comment thread docker-compose.yml Outdated
Comment thread build/default_configuration/adminer/plugins-enabled/enable-ssl.php
Comment thread build/default_configuration/certs/database/ca-key.pem
Comment thread docker-compose.yml Outdated
Comment thread build/default_configuration/adminer/plugins-enabled/enable-ssl;php.php Outdated
Comment thread readme.md Outdated
Comment thread readme.md Outdated
Comment thread addons/documentation/how_to.md
Comment thread addons/documentation/how_to.md
Comment thread build/default_configuration/adminer/plugins-enabled/enable-ssl.php
bdalsass and others added 9 commits May 5, 2026 07:28
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants