lowercase usernames

.gitea/workflows/lint.yaml

@@ -14,5 +14,5 @@ jobs:
           shellcheck --version
       - name: Run Linter
         run: |
-          shellcheck penv
+          shellcheck --exclude=SC1090,SC2046 penv
-          shellcheck jconf
+          shellcheck --exclude=SC1090,SC2046 jconf

jconf

@@ -39,59 +39,73 @@ if [ "$#" -lt 1 ]; then
     exit 1
 fi
 
+if [[ "$EUID" -ne 0 ]]; then
+    echo "Run the script as root"
+    exit 2
+fi
+
+
 cmd=$1
 case "$cmd" in
     adduser)
-
+        USER=$(echo "$2" | tr "[:upper:]" "[:lower:]")
-        exist=$(awk -F: -v username="$2" '{if ($1 == username && $3 >= 1000 && $1 != "nobody") {print 1; found=1; exit}} END {if (!found) print 0}' /etc/passwd)
+        exist=$(awk -F: -v username="$USER" '{if ($1 == username && $3 >= 1000 && $1 != "nobody") {print 1; found=1; exit}} END {if (!found) print 0}' /etc/passwd)
         if [ "$exist" -eq  1 ]; then
-            echo "$2 user already exists"
+            echo "$USER user already exists"
         else
-            while true; do
+            read -rp "Do you want to create an user account $USER [y/N]:" yn
-                read -p "Do you want to create an user account $2 [Y/n]:" yn
+            case $yn in
-                case $yn in
+                [Yy])
-                    [Yy]* ) 
+                    useradd "$USER"
-                        useradd $2;
+                    echo "Enter the password for $USER"
-                        echo "Enter the password for $2:";
+                    passwd "$USER"
-                        passwd $2
+                    echo "$USER user account created successfully"
-                        echo "$2 user account created successfully";
+                    ;;
-                        break;;
+                [Nn])
-                    [Nn]* ) exit 1;;
+                    exit 1
-                    * ) echo "Please answer Y or N.";;
+                    ;;
-                esac
+                * )
-            done
+                    echo "Please answer Y or N."
+                    ;;
+            esac
         fi
         exit 1
         ;;
-    
     deluser)
-
+        USER=$(echo "$2" | tr "[:upper:]" "[:lower:]")
-        exist=$(awk -F: -v username="$2" '{if ($1 == username && $3 >= 1000 && $1 != "nobody") {print 1; found=1; exit}} END {if (!found) print 0}' /etc/passwd)
+        exist=$(awk -F: -v username="$USER" '{if ($1 == username && $3 >= 1000 && $1 != "nobody") {print 1; found=1; exit}} END {if (!found) print 0}' /etc/passwd)
         if [ "$exist" -eq  0 ]; then
-            echo "$2 user doesn't exist"
+            echo "$USER user doesn't exist"
             exit 1
         else
-
+            read -rp "Do you want to delete $USER user account? [y/N]:" yn
-            while true; do
+            case $yn in
-                read -p "Do you want to delete $2 user account? [Y/n]:" yn
+                [Yy])
-                case $yn in
+                    ;;
-                    [Yy]* ) 
+                *)
-                        userdel -rRZ $2;
+                    exit 1
-                        echo "$2 user account deleted successfully";
+                    ;;
-                        break;;
+            esac
-                    [Nn]* ) exit 1;;
+            read -rp "This DELETES $USER's data, are you sure? [y/N]:" yn
-                    * ) echo "Please answer yes or no.";;
+            case $yn in
-                esac
+                [Yy])
-            done
+                    userdel -rRZ "$USER";
+                    echo "$USER user account deleted successfully"
+                    ;;
+                [Nn])
+                    exit 1
+                    ;;
+                * )
+                    echo "Please answer Y or N."
+                    ;;
+            esac
         fi
         exit 1
         ;;
-        
+    listusers|ls)
-    listusers)
+        husers=$(awk -F: '{ if ($3 >= 1000 && $1 != "nobody" ) {print $1}}' /etc/passwd)
-        husers= awk -F: '{ if ($3 >= 1000 && $1 != "nobody" ) {print $1}}' /etc/passwd 
+        echo "$husers"
-        echo $husers
         ;;
-    
     start)
         echo
         ;;

penv

@@ -43,7 +43,7 @@ create(){
         exit 2
     fi
     if [[ -d "$HOME/.envs/$name" ]]; then
-        echo Environment \"$name\" already exists.
+        echo Environment \""$name"\" already exists.
         exit 4
     fi
 
@@ -52,18 +52,18 @@ create(){
             echo "Base python3 not found on your system. Install it using your package manager"
             exit 3
         fi
-        vers=`/usr/bin/python3 -V | cut -d " " -f 2 | cut -d "." -f 1,2`
+        vers=$(/usr/bin/python3 -V | cut -d " " -f 2 | cut -d "." -f 1,2)
         echo "No version mentioned, using Python $vers"
     fi
-    if [ ! -f /usr/bin/python$vers ]; then
+    if [ ! -f /usr/bin/python"$vers" ]; then
         echo "Python $vers not installed on system."
         exit 7
     fi
-    mkdir -p $HOME/.envs
+    mkdir -p "$HOME"/.envs
-    /usr/bin/python$vers -m venv $HOME/.envs/$name
+    /usr/bin/python"$vers" -m venv "$HOME"/.envs/"$name"
-    source $HOME/.envs/$name/bin/activate
+    source "$HOME"/.envs/"$name"/bin/activate
     python -m pip install ipykernel ipywidgets > /dev/stderr
-    python -m ipykernel install --user --name $name > /dev/stderr
+    python -m ipykernel install --user --name "$name" > /dev/stderr
     echo
     echo "Created \"$name\". It should be available as a Jupyter Kernel."
     echo "Use \"pact $name\" to activate in command line."
@@ -81,12 +81,12 @@ delete(){
         echo "Run \"deactivate\" to deactivate it"
         exit 6
     fi
-    read -n 1 -p "Remove environment $name (y/N)? " choise
+    read -r -n 1 -p "Remove environment $name (y/N)? " choise
     case $choise in
     [yY])
-        source $HOME/.envs/$name/bin/activate
+        source "$HOME"/.envs/"$name"/bin/activate
         echo
-        echo Spec `jupyter-kernelspec remove -f -y $name`
+        echo Spec $(jupyter-kernelspec remove -f -y "$name")
         deactivate
         rm -r "$HOME/.envs/$name"
         echo "Data Removed $HOME/.envs/$name"
@@ -114,7 +114,7 @@ delete(){
 # }
 
 activate() {
-    echo Use \"pact $1\"
+    echo Use \"pact "$1"\"
 }
 
 list(){
@@ -122,7 +122,7 @@ list(){
         echo "PENV not initialized. Run \"penv init\""
         exit 8
     fi
-    ls $HOME/.envs
+    ls "$HOME"/.envs
 }
 
 install() {
@@ -130,8 +130,8 @@ install() {
         echo PENV already installed
         exit 10
     fi
-    cp $(realpath "$0") $HOME/.local/bin/
+    cp $(realpath "$0") "$HOME"/.local/bin/
-    mkdir -p $HOME/.envs/
+    mkdir -p "$HOME"/.envs/
     echo "
     pact () {
         if [[ -z \"\$1\" ]]; then
@@ -144,17 +144,17 @@ install() {
         fi
         . \$HOME/.envs/\$1/bin/activate
     }
-    " > $HOME/.envs/.penv.funcs
+    " > "$HOME"/.envs/.penv.funcs
-    grep -qxF "source \$HOME/.envs/.penv.funcs" $HOME/.bashrc || echo "source \$HOME/.envs/.penv.funcs" >> $HOME/.bashrc
+    grep -qxF "source \$HOME/.envs/.penv.funcs" "$HOME"/.bashrc || echo "source \$HOME/.envs/.penv.funcs" >> "$HOME"/.bashrc
     echo "Initialized PENV. Restart the shell"
 }
 
 uninstall() {
-    if [[ ! -z "$VIRTUAL_ENV" ]]; then
+    if [[ -n "$VIRTUAL_ENV" ]]; then
         echo "Deactivate the existing environment before uninstalling"
         exit 11
     fi
-    read -n 1 -p "REALLY UNINSTALL ALL THE ENVIRONMENTS?(y/N) " ans
+    read -r -n 1 -p "REALLY UNINSTALL ALL THE ENVIRONMENTS?(y/N) " ans
     case $ans in
         [yY])
             echo
@@ -170,9 +170,9 @@ uninstall() {
             return;
             ;;
     esac
-    sed -i "/source \$HOME\/.envs\/.penv.funcs/d" $HOME/.bashrc
+    sed -i "/source \$HOME\/.envs\/.penv.funcs/d" "$HOME"/.bashrc
-    rm -r $HOME/.envs
+    rm -r "$HOME"/.envs
-    rm $HOME/.local/bin/penv
+    rm "$HOME"/.local/bin/penv
     echo "Uninstalled PENV"
 }
 
@@ -186,16 +186,16 @@ cmd=$1
 
 case "$cmd" in
     new|mk)
-        create $2 $3
+        create "$2" "$3"
         ;;
     del|rm)
-        delete $2
+        delete "$2"
         ;;
     # setdef|sd)
     #     setdef $2
     #     ;;
     act|ac)
-        activate $2
+        activate "$2"
         ;;
     list|ls)
         list
@@ -212,7 +212,7 @@ case "$cmd" in
         ;;
     *)
         echo "invalid command: \"$1\"" >&2
-        echo $SCRIPT
+        echo "$SCRIPT"
         usage
         ;;
 esac