chore(docker): remove unnecessary commands from setup.sh (#2157)

* There's already an nginx user in the container
* Join chowns into a single statement
* Remove ``--ignore-scripts``, in an attempt to fix arm (unsuccessfully)

The chances of exploting ignore scripts (initially proposed by SonarCloud) is really low:

* It runs into a container
* The container is 99% of the times built in GiHub Actions environment.

It also introduces a mismatch between lint and typecheck commands and most dev environments (not everyone runs npm with that flag enabled).

packaging/docker/Dockerfile

@@ -14,7 +14,7 @@ COPY frontend /app/frontend
 WORKDIR /app/frontend
 
 # Install dependencies
-RUN npm ci --no-audit --ignore-scripts
+RUN npm ci --no-audit
 
 # Build client
 RUN COMMIT_HASH=$(git rev-parse HEAD) npm run build

packaging/docker/contents/postunpack.sh

@@ -15,14 +15,9 @@ rm -rf /usr/lib/libcrypto* /usr/lib/libintl* /usr/lib/libssl* \
     /usr/lib/engines-3 /usr/lib/modules-load.d /usr/lib/nginx /usr/lib/ossl-modules
 
 # CONTAINER ROOTLESS SETUP
-## Adding a nginx user that will manage the daemon
-adduser --system --shell /bin/false --no-create-home --disabled-login --disabled-password --gecos "nginx user" --group nginx
-## Set correct permissions
-chown nginx:nginx -R /var/cache/nginx
+### Set correct permissions and make frontend config.json file editable for the runtime user
 mkdir -p /run/nginx
-chown nginx:nginx -R /run/nginx
+chown nginx:nginx -R /run/nginx /var/cache/nginx /usr/share/nginx/html/config.json
 sed -i 's|/var/run|/var/run/nginx|g' $NGINX_CONFIG_FILE
-## Make frontend config.json file editable for the runtime user
-chown nginx:nginx -R /usr/share/nginx/html/config.json
 ## The 'user' config option is useless when running rootless and gives a warning
 sed -i '/^user /d' $NGINX_CONFIG_FILE