From e925016fb185af58b7161c1c0c5bb6b7d4742bba Mon Sep 17 00:00:00 2001
From: Davide Polonio <poloniodavide@gmail.com>
Date: Sat, 30 Jan 2021 14:49:10 +0000
Subject: [PATCH] Update setup script

---
 mail/setup.sh | 80 ++++++++++++++++++++++-----------------------------
 1 file changed, 34 insertions(+), 46 deletions(-)

diff --git a/mail/setup.sh b/mail/setup.sh
index 74ce1a7..b19c562 100755
--- a/mail/setup.sh
+++ b/mail/setup.sh
@@ -1,26 +1,22 @@
 #! /bin/bash
 
-# Wrapper for various setup scripts
-# included in docker-mailserver
+# version   v0.2.4 stable
+# executed  manually (via Make)
+# task      wrapper for various setup scripts
 
-SCRIPT='SETUP'
+SCRIPT='setup.sh'
 
 set -euEo pipefail
-trap '__log_err ${FUNCNAME[0]:-"?"} ${_:-"?"} ${LINENO:-"?"} ${?:-"?"}' ERR
+trap '__log_err ${FUNCNAME[0]:-"?"} ${BASH_COMMAND:-"?"} ${LINENO:-"?"} ${?:-"?"}' ERR
 trap '_unset_vars || :' EXIT
 
 function __log_err
 {
-  local FUNC_NAME LINE EXIT_CODE
-  FUNC_NAME="${1} / ${2}"
-  LINE="${3}"
-  EXIT_CODE="${4}"
-
   printf "\n––– \e[1m\e[31mUNCHECKED ERROR\e[0m\n%s\n%s\n%s\n%s\n\n" \
-    "  – script    = ${SCRIPT,,}.sh" \
-    "  – function  = ${FUNC_NAME}" \
-    "  – line      = ${LINE}" \
-    "  – exit code = ${EXIT_CODE}"
+    "  – script    = ${SCRIPT:-${0}}" \
+    "  – function  = ${1} / ${2}" \
+    "  – line      = ${3}" \
+    "  – exit code = ${4}" >&2
 }
 
 function _unset_vars
@@ -34,7 +30,7 @@ function _get_current_directory
 {
   if dirname "$(readlink -f "${0}")" &>/dev/null
   then
-    CDIR="$(cd "$(dirname "$(readlink -f "${0}")")" && pwd)"
+    CDIR="$(dirname "$(readlink -f "${0}")")"
   elif realpath -e -L "${0}" &>/dev/null
   then
     CDIR="$(realpath -e -L "${0}")"
@@ -48,7 +44,7 @@ _get_current_directory
 CRI=
 INFO=
 IMAGE_NAME=
-CONTAINER_NAME='mail'
+CONTAINER_NAME=
 DEFAULT_CONFIG_PATH="${CDIR}/config"
 USE_CONTAINER=false
 WISHED_CONFIG_PATH=
@@ -62,7 +58,7 @@ function _check_root
   if [[ ${EUID} -ne 0 ]]
   then
     echo "Curently docker-mailserver doesn't support podman's rootless mode, please run this script as root user."
-    return 1
+    exit 1
   fi
 }
 
@@ -101,15 +97,15 @@ function _inspect
 
 function _usage
 {
-  echo "${SCRIPT,,}.sh Bootstrapping Script
+  echo "${SCRIPT:-${0}} Bootstrapping Script
 
 Usage: ${0} [-i IMAGE_NAME] [-c CONTAINER_NAME] <subcommand> <subcommand> [args]
 
 OPTIONS:
 
-  -i IMAGE_NAME     The name of the docker-mailserver image, by default
-                    'tvial/docker-mailserver:latest' for docker, and
-                    'docker.io/tvial/docker-mailserver:latest' for podman.
+  -i IMAGE_NAME     The name of the docker-mailserver image
+                    The default value is
+                    'docker.io/mailserver/docker-mailserver:latest'
 
   -c CONTAINER_NAME The name of the running container.
 
@@ -146,7 +142,7 @@ SUBCOMMANDS:
 
   config:
 
-    ${0} config dkim <keysize> (default: 2048)
+    ${0} config dkim <keysize> (default: 4096) <domain> (optional - for LDAP systems)
     ${0} config ssl <fqdn>
 
   relay:
@@ -170,12 +166,8 @@ SUBCOMMANDS:
 
 function _docker_image_exists
 {
-  if ${CRI} history -q "${1}" >/dev/null 2>&1
-  then
-    return 0
-  else
-    return 1
-  fi
+  ${CRI} history -q "${1}" &>/dev/null
+  return ${?}
 }
 
 function _docker_image
@@ -204,17 +196,17 @@ function _docker_container
   then
     ${CRI} exec "${USE_TTY}" "${CONTAINER_NAME}" "${@}"
   else
-    echo "The docker-mailserver is not running!"
-    exit 5
+    echo "The mailserver is not running!"
+    exit 1
   fi
 }
 
 function _main
 {
-  if [[ -n $(command -v docker) ]]
+  if command -v docker &>/dev/null
   then
     CRI=docker
-  elif [[ -n $(command -v podman) ]]
+  elif command -v podman &>/dev/null
   then
     CRI=podman
     _check_root
@@ -223,29 +215,25 @@ function _main
     exit 10
   fi
 
-  INFO=$(${CRI} ps \
-    --no-trunc \
-    --format "{{.Image}};{{.Names}}" \
-    --filter label=org.label-schema.name="docker-mailserver" | \
-    tail -1)
+  INFO=$(${CRI} ps --no-trunc --format "{{.Image}};{{.Names}}" --filter \
+    label=org.opencontainers.image.title="docker-mailserver" | tail -1)
 
   IMAGE_NAME=${INFO%;*}
   CONTAINER_NAME=${INFO#*;}
 
   if [[ -z ${IMAGE_NAME} ]]
   then
-    if [[ ${CRI} == "docker" ]]
-    then
-      IMAGE_NAME=tvial/docker-mailserver:latest
-    elif [[ ${CRI} == "podman" ]]
-    then
-      IMAGE_NAME=docker.io/tvial/docker-mailserver:latest
-    fi
+    IMAGE_NAME=${NAME:-docker.io/mailserver/docker-mailserver:latest}
   fi
 
-  if tty -s
+  if test -t 0
   then
     USE_TTY="-ti"
+  else
+    # GitHub Actions will fail (or really anything else
+    #   lacking an interactive tty) if we don't set a
+    #   value here; "-t" alone works for these cases.
+    USE_TTY="-t"
   fi
 
   local OPTIND
@@ -329,13 +317,13 @@ function _main
       shift ; case ${1:-} in
         set      ) shift ; _docker_image setquota "${@}" ;;
         del      ) shift ; _docker_image delquota "${@}" ;;
-        *        )   _usage ;;
+        *        ) _usage ;;
       esac
       ;;
 
     config)
       shift ; case ${1:-} in
-        dkim     ) _docker_image generate-dkim-config "${2:-2048}" ;;
+        dkim     ) _docker_image generate-dkim-config "${2:-4096}" "${3:-}" ;;
         ssl      ) _docker_image generate-ssl-certificate "${2}" ;;
         *        ) _usage ;;
       esac