docker-geonetwork icon indicating copy to clipboard operation
docker-geonetwork copied to clipboard

Support for gn_db env variable added

Open Zalitoar opened this issue 7 years ago • 1 comments

If gn_db isn't defined when run command is executed 'geonetwork' is used as default database name. 'db_admin' variable was deleted, it isn't used by the application.

Zalitoar avatar May 19 '17 17:05 Zalitoar

Quick question as I was about to implement the same change (and this PR is already over a year old): Why are you not using the same "default value" syntax as is already used for other variables e.g.

db_host="${POSTGRES_DB_HOST:-postgres}"

I'd suggest something like:

db_gn="${POSTGRES_DB_GN:-geonetwork}"

grmpfhmbl avatar Oct 23 '18 20:10 grmpfhmbl