Update start.sh

[alexkutsan]

On NIX OS there is no /bin/bash.
It located in

▶ where bash
/run/current-system/sw/bin/bash

But /usr/bin/env should be on most of Linux distros

[evertramos]

Yeah.. I being using this in all other scripts and forgot to update this one. Thanks!

[teslasoft-de]

Guys, can you please explain why you are:

• accepting a change for an insecure shebang statement?
  (sorry but different bash versions on one host in the age of docker is nonsense)
  Searching for bash in env is evil. If I would wear a black hat then all your base are belong to us...
• changing the shebang of only one sh file, why not all other files too? (Please don't do it)

Please don't take me wrong, Docker-Compose-Letsencrypt is one of the coolest projects ever.
I LOVE IT!
But it gets frustrating if things stop working after an update...

[evertramos]

Hello @teslasoft-de!

You have your point of view which I agree in part as I have used FreeBSD and even Solaris in the past which we would need this portability.

I am pushing these docker scripts to a whole new level and will make it public in a few month now. I will keep this updated with this version. I have being working too much due to covid... as most of us.

Let's discuss the best option to accomplish those who need this portability. No need it for myself these days, once all my sever's images are well customized using regular bash, so I would be glad to use only /bin/bash shebang.

Thanks for you message and your concern.

What do you guy think?

@j7an, @fracz