kartoza / docker-postgis

Dockerfile for postgis
GNU General Public License v2.0
648 stars 317 forks source link

Fix broken tuning params for conf when using timescale tuning #416

Closed NyakudyaA closed 1 year ago

NyakudyaA commented 1 year ago

What is the bug or the crash?

In https://github.com/kartoza/docker-postgis/blob/develop/scripts/setup-conf.sh#L107, the logic seems to allow to overwrite the default PostgreSQL conf. This is not ideal. An alternative should be to write a new file and use directory includes.

It also seems https://github.com/kartoza/docker-postgis/blob/develop/scripts/setup-conf.sh#L109 is never reached in execution

Steps to reproduce the issue

Check the source code

Versions

PG timescale

Additional context

Fix the issue