postfixadmin / docker

Docker Image for PostfixAdmin
https://hub.docker.com/_/postfixadmin
GNU General Public License v2.0
73 stars 21 forks source link

POSTFIXADMIN_DB_NAME_FILE does nothing #67

Open Arch-Impuls opened 1 year ago

Arch-Impuls commented 1 year ago

From documentation :

All environment vars are supporting the secret's docker strategy.

From docker-entrypoint.sh :

Init vars for running script

POSTFIXADMIN_DB_TYPE=$(get_env_value 'POSTFIXADMIN_DB_TYPE' 'sqlite') POSTFIXADMIN_DB_HOST=$(get_env_value "POSTFIXADMIN_DB_HOST" "") POSTFIXADMIN_DB_PORT=$(get_env_value "POSTFIXADMIN_DB_PORT" "") POSTFIXADMIN_DB_USER=$(get_env_value "POSTFIXADMIN_DB_USER" "") POSTFIXADMIN_DB_PASSWORD=$(get_env_value "POSTFIXADMIN_DB_PASSWORD" "") POSTFIXADMIN_SMTP_SERVER=$(get_env_value "POSTFIXADMIN_SMTP_SERVER" "localhost") POSTFIXADMIN_SMTP_PORT=$(get_env_value "POSTFIXADMIN_SMTP_PORT" "25") POSTFIXADMIN_ENCRYPT=$(get_env_value "POSTFIXADMIN_ENCRYPT" "md5crypt")

DEFAULT_SETUP_PASSWORD="changeme" POSTFIXADMIN_SETUP_PASSWORD=$(get_env_value "POSTFIXADMIN_SETUP_PASSWORD" "${DEFAULT_SETUP_PASSWORD}")

Actually, POSTFIXADMIN_DB_NAME_FILE isn't working properly because the line to handle it is missing from the entrypoint script POSTFIXADMIN_DB_NAME =$(get_env_value "POSTFIXADMIN_DB_NAME" "")

pifeifei commented 9 months ago

+1