From b808458daa86105dd2101893961338912961ee92 Mon Sep 17 00:00:00 2001
From: GitLab Bot <gitlab-bot@gitlab.com>
Date: Tue, 9 Nov 2021 12:12:15 +0000
Subject: Add latest changes from gitlab-org/gitlab@master

---
 bin/background_jobs      | 18 +++++++++---------
 bin/mail_room            |  9 ++++-----
 bin/parallel-rsync-repos |  6 +++---
 bin/web                  | 10 +++++-----
 bin/with_env             |  1 +
 5 files changed, 22 insertions(+), 22 deletions(-)

(limited to 'bin')

diff --git a/bin/background_jobs b/bin/background_jobs
index d8929881f12..f301bb46ca9 100755
--- a/bin/background_jobs
+++ b/bin/background_jobs
@@ -1,12 +1,12 @@
 #!/usr/bin/env bash
 
-cd $(dirname $0)/..
+cd "$(dirname "$0")/.." || exit
+
 app_root=$(pwd)
 sidekiq_workers=${SIDEKIQ_WORKERS:-1}
 sidekiq_queues=${SIDEKIQ_QUEUES:-*} # Queues to listen to; default to `*` (all)
 sidekiq_pidfile="$app_root/tmp/pids/sidekiq-cluster.pid"
 sidekiq_logfile="$app_root/log/sidekiq.log"
-gitlab_user=$(ls -l config.ru | awk '{print $3}')
 
 trap cleanup EXIT
 
@@ -17,26 +17,26 @@ warn()
 
 get_sidekiq_pid()
 {
-  if [ ! -f $sidekiq_pidfile ]; then
+  if [ ! -f "$sidekiq_pidfile" ]; then
     warn "No pidfile found at $sidekiq_pidfile; is Sidekiq running?"
     return
   fi
 
-  cat $sidekiq_pidfile
+  cat "$sidekiq_pidfile"
 }
 
 stop()
 {
   sidekiq_pid=$(get_sidekiq_pid)
 
-  if [ $sidekiq_pid ]; then
-    kill -TERM $sidekiq_pid
+  if [ "$sidekiq_pid" ]; then
+    kill -TERM "$sidekiq_pid"
   fi
 }
 
 restart()
 {
-  if [ -f $sidekiq_pidfile ]; then
+  if [ -f "$sidekiq_pidfile" ]; then
     stop
   fi
 
@@ -53,12 +53,12 @@ start_sidekiq()
   fi
 
   # sidekiq-cluster expects an argument per process.
-  for (( i=1; i<=$sidekiq_workers; i++ ))
+  for (( i=1; i<=sidekiq_workers; i++ ))
   do
     processes_args+=("${sidekiq_queues}")
   done
 
-  ${cmd} bin/sidekiq-cluster "${processes_args[@]}" -P $sidekiq_pidfile -e $RAILS_ENV "$@" 2>&1 | tee -a $sidekiq_logfile
+  ${cmd} bin/sidekiq-cluster "${processes_args[@]}" -P "$sidekiq_pidfile" -e "$RAILS_ENV" "$@" 2>&1 | tee -a "$sidekiq_logfile"
 }
 
 cleanup()
diff --git a/bin/mail_room b/bin/mail_room
index cf9d422909e..3717e49e37f 100755
--- a/bin/mail_room
+++ b/bin/mail_room
@@ -1,6 +1,6 @@
 #!/bin/sh
 
-cd $(dirname $0)/.. || exit 1
+cd "$(dirname "$0")/.." || exit 1
 app_root=$(pwd)
 
 mail_room_pidfile="$app_root/tmp/pids/mail_room.pid"
@@ -9,8 +9,7 @@ mail_room_config="$app_root/config/mail_room.yml"
 
 get_mail_room_pid()
 {
-  local pid
-  pid=$(cat $mail_room_pidfile)
+  pid=$(cat "$mail_room_pidfile")
   if [ -z "$pid" ] ; then
     echo "Could not find a PID in $mail_room_pidfile"
     exit 1
@@ -20,13 +19,13 @@ get_mail_room_pid()
 
 start()
 {
-  bin/daemon_with_pidfile $mail_room_pidfile bundle exec mail_room --log-exit-as json -q -c $mail_room_config >> $mail_room_logfile 2>&1
+  bin/daemon_with_pidfile "$mail_room_pidfile" bundle exec mail_room --log-exit-as json -q -c "$mail_room_config" >> "$mail_room_logfile" 2>&1
 }
 
 stop()
 {
   get_mail_room_pid
-  kill -TERM $mail_room_pid
+  kill -TERM "$mail_room_pid"
 }
 
 restart()
diff --git a/bin/parallel-rsync-repos b/bin/parallel-rsync-repos
index 21921148fa0..bd849371766 100755
--- a/bin/parallel-rsync-repos
+++ b/bin/parallel-rsync-repos
@@ -32,20 +32,20 @@ if [ -z "$RSYNC" ] ; then
   RSYNC=rsync
 fi
 
-if ! cd $SRC ; then
+if ! cd "$SRC" ; then
   echo "cd $SRC failed"
   exit 1
 fi
 
 rsyncjob() {
-  relative_dir="./${1#$SRC}"
+	relative_dir="./${1#"$SRC"}"
 
   if ! $RSYNC --delete --relative -a "$relative_dir" "$DEST" ; then
     echo "rsync $1 failed"
     return 1
   fi
 
-  echo "$1" >> $LOGFILE
+  echo "$1" >> "$LOGFILE"
 }
 
 export LOGFILE SRC DEST RSYNC
diff --git a/bin/web b/bin/web
index c1ab4718f0d..4d2a16f6665 100755
--- a/bin/web
+++ b/bin/web
@@ -2,7 +2,7 @@
 
 set -e
 
-cd $(dirname $0)/..
+cd "$(dirname "$0")/.."
 app_root=$(pwd)
 
 puma_pidfile="$app_root/tmp/pids/puma.pid"
@@ -25,12 +25,12 @@ get_puma_pid()
 
 start()
 {
-  spawn_puma &
+  spawn_puma "$@" &
 }
 
 start_foreground()
 {
-  spawn_puma
+  spawn_puma "$@"
 }
 
 stop()
@@ -46,10 +46,10 @@ reload()
 
 case "$1" in
   start)
-    start
+    start "$@"
     ;;
   start_foreground)
-    start_foreground
+    start_foreground "$@"
     ;;
   stop)
     stop
diff --git a/bin/with_env b/bin/with_env
index e678fa2f0cc..b0647a50e27 100755
--- a/bin/with_env
+++ b/bin/with_env
@@ -10,6 +10,7 @@ shift
 
 # Use set -a to export all variables defined in env_file.
 set -a
+# shellcheck disable=SC1090
 . "${env_file}"
 set +a
 
-- 
cgit v1.2.3