nextcloud / nextcloud/docker

Add dynamic config for Postgres SSL mode

Open
#2,477 2 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

0. Needs triage bug feature: auto config (environment variables)
Dominant language
Shell
Stars
7.4k
Forks
1.9k
PR merge metrics
No merged PRs in 30d

Description

Latest release nextcloud:30.0.16-fpm (see https://github.com/nextcloud/server/pull/55170) brings postgresql SSL support but the docker image does not provide dynamic config.

Example: 30/fpm/config/pgsql_ssl.config.php

<?php
if (getenv('POSTGRES_SSL_MODE')) {
  $CONFIG = array (
    'pgsql_ssl' => array(
      'mode' => getenv('POSTGRES_SSL_MODE'),  // see https://www.postgresql.org/docs/current/libpq-connect.html#LIBPQ-CONNECT-SSLMODE
      'rootcert' => getenv('POSTGRES_SSL_ROOTCERT') ?: null,
      ...
    ),
  );
}

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by inspecting the 30/fpm/config directory and the requested pgsql_ssl.config.php path, then review the PostgreSQL SSL mode documentation linked in the issue. Done means the Docker image provides dynamic PostgreSQL SSL configuration using POSTGRES_SSL_MODE and the related root certificate setting for the Nextcloud 30.0.16-fpm release.

Written by the indexing model from the issue text.

Assessment

Tech stack
docker, php, postgresql
Domain
databases, devops
Issue type
Feature
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
52/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.