Require --dangerWillRobinson for removing >1 config

MetPX · Oct 18, 2023 · 72c62b8 · 72c62b8
1 parent c2e4a94
commit 72c62b8
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 0 deletions.
diff --git a/bin/sr3_action_remove b/bin/sr3_action_remove
@@ -21,6 +21,9 @@ for argu in "$@"; do
     if [[ "$argu" == "remove" ]] ; then
         continue
     fi
+    if [[ "$argu" == "--dangerWillRobinson" ]] ; then
+        continue
+    fi
     files_to_remove+=("${pump_path}${argu}.conf")
 done
 

diff --git a/bin/sr3d b/bin/sr3d
@@ -13,5 +13,10 @@ sr3r "sr3 $*"
 
 # Special cases for certain actions AFTER running on nodes
 if [[ "$action" == "remove" ]] ; then
+    # Removing more than one config at a time requires --dangerWillRobinson
+    if [[ "$*" != *"--dangerWillRobinson"* ]] && [[ $# -gt 2 ]]; then
+        echo "[ERROR] --dangerWillRobinson is required to remove more than one config."
+        exit 1
+    fi
     sr3_action_remove "$@"
 fi
diff --git a/completion/sr3_tools_completion.bash b/completion/sr3_tools_completion.bash
@@ -83,6 +83,7 @@ component_config()
 # sr3d syntax is the same as sr3:
 #     sr3d action [component/config]
 # except some actions (edit, foreground, etc.) don't make sense on a cluster
+# TODO: support --dangerWillRobinson as 1st arg (before action) or 2nd arg (after action)
 ###
 _sr3d()
 {