From bce5827f64e98d9ee0925930573de2643cce7283 Mon Sep 17 00:00:00 2001
From: Evgenii Stratonikov <e.stratonikov@yadro.com>
Date: Thu, 2 Mar 2023 14:38:00 +0300
Subject: [PATCH] [#83] pre-commit: Add shellcheck hook

Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>
---
 .pre-commit-config.yaml         | 6 ++++++
 debian/frostfs-ir.postinst      | 6 +++---
 debian/frostfs-storage.postinst | 6 +++---
 3 files changed, 12 insertions(+), 6 deletions(-)

diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
index ce4709283..a58352f88 100644
--- a/.pre-commit-config.yaml
+++ b/.pre-commit-config.yaml
@@ -28,3 +28,9 @@ repos:
     hooks:
       - id: gitlint
         stages: [commit-msg]
+
+  - repo: https://github.com/koalaman/shellcheck-precommit
+    rev: v0.9.0
+    hooks:
+    -   id: shellcheck
+#       args: ["--severity=warning"]  # Optionally only show errors and warnings
diff --git a/debian/frostfs-ir.postinst b/debian/frostfs-ir.postinst
index cb4a10310..eb9d381c9 100755
--- a/debian/frostfs-ir.postinst
+++ b/debian/frostfs-ir.postinst
@@ -28,9 +28,9 @@ case "$1" in
             chmod -f 0640 /etc/frostfs/$USERNAME/config.yml || true
             chmod -f 0640 /etc/frostfs/$USERNAME/control.yml || true
         fi
-        USERDIR=$(getent passwd "frostfs-$USERNAME" | cut -d: -f6)
-        if ! dpkg-statoverride --list frostfs-$USERDIR >/dev/null; then
-            chown -f frostfs-$USERNAME: $USERDIR
+        USERDIR="$(getent passwd frostfs-$USERNAME | cut -d: -f6)"
+        if ! dpkg-statoverride --list frostfs-"$USERDIR" >/dev/null; then
+            chown -f frostfs-$USERNAME: "$USERDIR"
         fi
     ;;
 
diff --git a/debian/frostfs-storage.postinst b/debian/frostfs-storage.postinst
index 0ecce7445..88fa53be5 100755
--- a/debian/frostfs-storage.postinst
+++ b/debian/frostfs-storage.postinst
@@ -28,9 +28,9 @@ case "$1" in
             chmod -f 0640 /etc/frostfs/$USERNAME/config.yml || true
             chmod -f 0640 /etc/frostfs/$USERNAME/control.yml || true
         fi
-        USERDIR=$(getent passwd "frostfs-$USERNAME" | cut -d: -f6)
-        if ! dpkg-statoverride --list frostfs-$USERDIR >/dev/null; then
-            chown -f frostfs-$USERNAME: $USERDIR
+        USERDIR=$(getent passwd frostfs-$USERNAME | cut -d: -f6)
+        if ! dpkg-statoverride --list frostfs-"$USERDIR" >/dev/null; then
+            chown -f frostfs-$USERNAME: "$USERDIR"
         fi
         USERDIR=/srv/frostfs
         if ! dpkg-statoverride --list frostfs-$USERDIR >/dev/null; then