Docker: various updates

- use exec in entrypoint.sh, get rid of extra shell process
- use CMD instead of ENTRYPOINT, so image can be run with /bin/sh arg

docker/Dockerfile

@@ -1,14 +1,16 @@
 FROM alpine:3.6
 
-RUN apk add --no-cache --update apache2-utils
+ENV DATA_DIRECTORY /data
-COPY docker/*_user rest-server /usr/bin/
+ENV PASSWORD_FILE /data/.htpasswd
-COPY docker/entry.sh /
 
-ENV DATA_DIRECTORY=/data
+RUN apk add --no-cache --update apache2-utils
-ENV PASSWORD_FILE=/data/.htpasswd
+
+COPY rest-server docker/*_user /usr/bin/
 
 VOLUME /data
 
 EXPOSE  80
 
-ENTRYPOINT ["/entry.sh"]
+COPY docker/entrypoint.sh /entrypoint.sh
+
+CMD [ "/entrypoint.sh" ]

docker/entry.sh

@@ -1,21 +0,0 @@
-#!/bin/sh
-
-set -e
-
-mkdir -p $DATA_DIRECTORY
-
-if [ -z "$DISABLE_AUTHENTICATION" ]; then
-    if [ ! -f $PASSWORD_FILE ]; then
-        touch $PASSWORD_FILE
-    fi
-
-    if [ ! -s $PASSWORD_FILE ]; then
-        echo
-        echo "**WARNING** No user exists, please 'docker exec -ti \$CONTAINER_ID create_user'"
-        echo
-    fi
-else
-    rm -f $PASSWORD_FILE
-fi
-
-rest-server --listen ":80" $OPTIONS --path $DATA_DIRECTORY

docker/entrypoint.sh

@@ -0,0 +1,19 @@
+#!/bin/sh
+
+set -e
+
+if [ -z "$DISABLE_AUTHENTICATION" ]; then
+    if [ ! -f "$PASSWORD_FILE" ]; then
+        touch "$PASSWORD_FILE"
+    fi
+
+    if [ ! -s "$PASSWORD_FILE" ]; then
+        echo
+        echo "**WARNING** No user exists, please 'docker exec -it \$CONTAINER_ID create_user'"
+        echo
+    fi
+else
+    rm -f "$PASSWORD_FILE"
+fi
+
+exec rest-server --listen ":80" $OPTIONS --path "$DATA_DIRECTORY"