nextcloud / docker

⛴ Docker image of Nextcloud
https://hub.docker.com/_/nextcloud/
GNU Affero General Public License v3.0
5.95k stars 1.82k forks source link

Support setting database port #2300

Open jessebot opened 2 hours ago

jessebot commented 2 hours ago

Hoi nextcloud/docker friends!

I wanted to inquiry about adding some env vars for database ports? Happy to submit the PR for setting postgresql/mysql ports via env vars, but I'm not entirely sure if it's supported upstream in nextcloud/server. Either way, I wanted to kick off the conversation here, so we can then maybe support it upstream in nextcloud/helm.

Please also see https://github.com/nextcloud/helm/issues/632 for more info.

Thanks and kind regards!

joshtrichards commented 2 hours ago

The port is just specified as part of the dbhost parameter in Nextcloud. I don't believe there's anything specific we need to do to support it in the image.

I haven't used an alternative port admittedly that I can recall, but I do use UNIX sockets a fair bit in my deploys. They're specified in a similar manner and I know MYSQL_HOST and POSTGRES_HOST already work well with the syntax for that.