Quote variables in entrypoint.sh test conditions

FORCE_REINIT is not set in .env.sample (only referenced in a comment), so
on a normal start '[ ${FORCE_REINIT} = "true" ]' runs as '[ = "true" ]'
and errors with '[: =: unary operator expected'. The || chain falls through
to the lock-file check so init still works, but the error is printed on
every startup and the toggle is fragile. Quote the compared variables (also
INVOKE_LOG_STDOUT and IS_CELERY) so an empty value compares cleanly.

Author: Valyrian-Code

src/entrypoint.sh

@@ -5,7 +5,7 @@ set -e
 
 INVOKE_LOG_STDOUT=${INVOKE_LOG_STDOUT:-FALSE}
 invoke () {
-    if [ $INVOKE_LOG_STDOUT = 'true' ] || [ $INVOKE_LOG_STDOUT = 'True' ]
+    if [ "$INVOKE_LOG_STDOUT" = 'true' ] || [ "$INVOKE_LOG_STDOUT" = 'True' ]
     then
         /usr/local/bin/invoke $@
     else
@@ -29,15 +29,15 @@ source $HOME/.override_env
 
 cmd="$@"
 
-if [ ${IS_CELERY} = "true" ]  || [ ${IS_CELERY} = "True" ]
+if [ "${IS_CELERY}" = "true" ]  || [ "${IS_CELERY}" = "True" ]
 then
     echo "Executing Celery server $cmd for Production"
 else
 
     invoke migrations
     invoke prepare
 
-    if [ ${FORCE_REINIT} = "true" ]  || [ ${FORCE_REINIT} = "True" ] || [ ! -e "/mnt/volumes/statics/geonode_init.lock" ]; then
+    if [ "${FORCE_REINIT}" = "true" ]  || [ "${FORCE_REINIT}" = "True" ] || [ ! -e "/mnt/volumes/statics/geonode_init.lock" ]; then
         invoke fixtures
         invoke initialized
         invoke updateadmin