Delete pad after delay

Hello! J'aimerais bien une relecture, surtout de la part des admins actuels. Je doute surtout du fait que ma nouvelle version du docker-compose ne casse pas la prod (conservation des noms de volumes etc). J'ai tout vérifié mais on n'est jamais trop sûrs ;)

Au passage, j'ai aussi:

• accéléré la compilation de l'image en fixant les pb de chown
• bougé la variable d'environnement de Abiword dans le dockerfile, puisque c'est le dockerfile qui contrôle le path (actuellement c'est le fait qu'on utilise debian/apt qui détermine l'endroit où sera installé abiword). Mais ça reste une variable d'environnement, qui peut être overridée dans le docker-compose si nécessaire.
• normalisé les noms là où c'était possible (j'espère ne pas avoir touché aux noms réels des volumes)

@qduchemi je t'assigne de façon totalement arbitraire, hésite pas à refiler le bébé à quelqu'un d'autre, J'EN APPELLE AU DISCERNEMENT DE CHACUN

pica-etherpad/Dockerfile

@@ -13,6 +13,9 @@ ARG ETHERPAD_PLUGINS="\
   ep_pads_stats ep_prompt_for_name ep_set_title_on_pad \
   ep_subscript_and_superscript"
 
+
+
+
 RUN apt-get update && \
     apt-get install -y \
       curl \
@@ -25,15 +28,27 @@ RUN apt-get update && \
       unzip && \
     mkdir -p /opt/etherpad-lite/
 
+# Try to stick to the LDAP UID for etherpad (OU=services)
+# WARNING: is also defined hereinbelow
+
+ARG UID=5004
+
+RUN useradd --uid ${UID} --create-home etherpad
+
+RUN chown etherpad /opt/etherpad-lite
+
 WORKDIR /opt/etherpad-lite
-# Install Etherpad
+
+# Install Etherpad, as user etherpad
+USER etherpad
+
 RUN curl -SL https://github.com/ether/etherpad-lite/archive/${ETHERPAD_VERSION_BUILD}.zip > etherpad.zip && \
     unzip etherpad && \
     rm -Rf etherpad.zip etherpad-lite-${ETHERPAD_VERSION_BUILD}/.git && \
     mv etherpad-lite-${ETHERPAD_VERSION_BUILD}/* . && \
     bin/installDeps.sh
 
-COPY ./landing-page ./src/templates/landing-page
+COPY --chown=etherpad ./landing-page ./src/templates/landing-page
 
 # Build and replace landing page
 RUN npm install minify && \
@@ -51,29 +66,40 @@ RUN for PLUGIN_NAME in ${ETHERPAD_PLUGINS}; do npm install "${PLUGIN_NAME}"; don
 FROM base
 
 # Try to stick to the LDAP UID for etherpad (OU=services)
+# WARNING: is also defined hereinabove
+
 ARG UID=5004
-COPY entrypoint.sh /opt/etherpad-lite/entrypoint.sh
 
 RUN useradd --uid ${UID} --create-home etherpad
 
+# Used to avoid very long chown (due to https://github.com/docker/for-linux/issues/388)
+USER etherpad
+
 # Import des fichiers téléchargés dans l'image downloader
 COPY --from=downloader /opt/etherpad-lite /opt/etherpad-lite/
-RUN chown -R etherpad /opt/etherpad-lite
 
+# Install debian packages, as root
+USER root
 RUN apt-get update && \
     apt-get install -y curl nano postgresql-client abiword && \
-    chmod +x /opt/etherpad-lite/entrypoint.sh && \
-    usermod -d /opt/etherpad-lite etherpad && \
-    chown -R etherpad /opt/etherpad-lite && \
     rm -rf /var/lib/apt/lists/*
 
-RUN mkdir /opt/etherpad-lite/deleted_pads && \
-    chown -R etherpad:etherpad /opt/etherpad-lite/deleted_pads
+# Set etherpad as owner of etherpad (permission for its content was already set in downloader)
+RUN chown etherpad:etherpad /opt/etherpad-lite
 
+# Copy entrypoint script
+COPY entrypoint.sh /opt/etherpad-lite/entrypoint.sh
+RUN chmod +x /opt/etherpad-lite/entrypoint.sh && \
+	usermod -d /opt/etherpad-lite etherpad
+
+# Create path owned by etherpad to mount the API key
+RUN mkdir /mountedAPIkey && chown etherpad:etherpad /mountedAPIkey
+
+# Settings for entrypoint
+USER etherpad
 EXPOSE 8080
 HEALTHCHECK --interval=20s --timeout=3s CMD curl --fail http://localhost:8080 || exit 1
 WORKDIR /opt/etherpad-lite/
-USER etherpad
-
-ENTRYPOINT ["/opt/etherpad-lite/entrypoint.sh"]
-CMD [ "/bin/bash", "-c", "/opt/etherpad-lite/bin/run.sh" ]
+# Installed here by debian
+ENV ABIWORD "/usr/bin/abiword"
+ENTRYPOINT ["/opt/etherpad-lite/entrypoint.sh", "/opt/etherpad-lite/bin/run.sh" ]