Securitry: Pin bash in dockerfile

[helge000]

Describe the Bug

The current Dockerfile fails build check because of missing JSON args, failing my CI. To fix this, we should explicitly set bash, as we need shell for command chaining.

Check complete, 1 warning has been found!

WARNING: JSONArgsRecommended - https://docs.docker.com/go/dockerfile/rule/json-args-recommended/
JSON arguments recommended for CMD to prevent unintended behavior related to OS signals
Dockerfile:51
--------------------
  50 |     
  51 | >>> CMD : \
  52 | >>>      && node /directus/cli.js bootstrap \
  53 | >>>      && node /directus/cli.js start \
  54 | >>>      ;
  55 |     
--------------------

To Reproduce

Run docker build --check .

Directus Version

v9.11.0.1

Hosting Strategy

Self-Hosted (Docker Image)

[helge000]

I chose the SHELL workaround because of less impact. The other workaround using a wrapper is more elegant, I guess? Please let me know; I will modify it.

[ldechamps]

I’m closing the issue because the problem has been resolved.

[helge000]

Well, while I was relying on CI I did not catch that /bin/bash is not availabe in the image since it is based on busybox. We have to use /bin/sh. Sorry for that - 😳
PR incomming (I have no permisson to reopen the issue)