address feedback

Author: bangbingsyb

action.yml

@@ -10,13 +10,13 @@ inputs:
   nginx-deployment-name:
     description: 'The name of the NGINX deployment'
     required: true
-  nginx-config-file-path:
-    description: 'The file path of the NGINX configuration file in the Git repository'
+  nginx-config-relative-file-path:
+    description: 'The relative file path of the NGINX configuration file in the Git repository'
     required: true
     default: './config/nginx.conf'
 runs:
   using: "composite"
   steps:
     - name: 'Deploy configuration to the NGINX deployment in Azure'
-      run: ${{github.action_path}}/src/deploy-config.sh ${{ inputs.subscription-id }} ${{ inputs.resource-group-name }} ${{ inputs.nginx-deployment-name }} ${{ inputs.nginx-config-file-path }}
+      run: ${{github.action_path}}/src/deploy-config.sh ${{ inputs.subscription-id }} ${{ inputs.resource-group-name }} ${{ inputs.nginx-deployment-name }} ${{ inputs.nginx-config-relative-file-path }}
       shell: bash

src/deploy-config.sh

@@ -1,7 +1,6 @@
 #!/bin/bash
-
-# Stop on error.
-set -e
+set -euo pipefail
+IFS=$'\n\t'
 
 subscriptionId=$1
 resourceGroupName=$2
@@ -11,12 +10,10 @@ nginxConfigurationFile=$4
 # Read and encode the NGINX configuration file content.
 if [ -f "$nginxConfigurationFile" ]
 then
-    echo "NGINX configuration"
-    cat "$nginxConfigurationFile"
-    echo ""
+    echo "The NGINX configuration file was found."
 else 
-    echo "NGINX configuration $nginxConfigurationFile does not exist."
-    exit 32
+    echo "The NGINX configuration file $nginxConfigurationFile does not exist."
+    exit 2
 fi
 
 encodedConfigContent=$(base64 $nginxConfigurationFile)