Browse Source

fix shellcheck warnings

neil 2 years ago
parent
commit
e275cb1efd
3 changed files with 3 additions and 3 deletions
  1. 1 1
      .github/workflows/shellcheck.yml
  2. 1 1
      deploy/gitlab.sh
  3. 1 1
      dnsapi/dns_dynv6.sh

+ 1 - 1
.github/workflows/shellcheck.yml

@@ -26,7 +26,7 @@ jobs:
     - name: Install Shellcheck
     - name: Install Shellcheck
       run: sudo apt-get install -y shellcheck
       run: sudo apt-get install -y shellcheck
     - name: DoShellcheck
     - name: DoShellcheck
-      run: shellcheck -V  && shellcheck -e SC2181 **/*.sh && echo "shellcheck OK"
+      run: shellcheck -V  && shellcheck -e SC2181 -e SC2089 **/*.sh && echo "shellcheck OK"
 
 
   shfmt:
   shfmt:
     runs-on: ubuntu-latest
     runs-on: ubuntu-latest

+ 1 - 1
deploy/gitlab.sh

@@ -67,7 +67,7 @@ gitlab_deploy() {
 
 
   error_response="error"
   error_response="error"
 
 
-  if test ${_response#*"$error_response"} != "$_response"; then
+  if test "${_response#*"$error_response"}" != "$_response"; then
     _err "Error in deploying certificate:"
     _err "Error in deploying certificate:"
     _err "$_response"
     _err "$_response"
     return 1
     return 1

+ 1 - 1
dnsapi/dns_dynv6.sh

@@ -94,7 +94,7 @@ _get_domain() {
   _your_hosts="$(echo "$_your_hosts" | awk '/\./ {print $1}')"
   _your_hosts="$(echo "$_your_hosts" | awk '/\./ {print $1}')"
   for l in $_your_hosts; do
   for l in $_your_hosts; do
     #echo "host: $l"
     #echo "host: $l"
-    if test ${_full_domain#*"$l"} != "$_full_domain"; then
+    if test "${_full_domain#*"$l"}" != "$_full_domain"; then
       _record=${_full_domain%."$l"}
       _record=${_full_domain%."$l"}
       _host=$l
       _host=$l
       _debug "The host is $_host and the record $_record"
       _debug "The host is $_host and the record $_record"