Merge pull request #64 from PeterDaveHello/shfmt

Format shell scripts with shfmt and test it on CI

.travis.yml

@@ -7,8 +7,18 @@ language: minimal
 git:
   depth: 5
 
+jobs:
+  include:
+    - stage: Test
+      name: Docker build
       script:
         - docker build -t dnscrypt-server-docker-ci-test .
+    - stage: Test
+      name: Check shell script format
+      script:
+        - docker run -it --rm -v "$(pwd)":/sh -w /sh peterdavehello/shfmt:2.6.4 shfmt -sr -i 4 -l -w -ci .
+        - git diff --color
+        - git diff --stat=220 --color --exit-code
 
 services:
   - docker

entrypoint.sh

@@ -24,20 +24,23 @@ init() {
     case "$provider_name" in
         .*) usage ;;
         2.dnscrypt-cert.*) ;;
-        *) provider_name="2.dnscrypt-cert.${provider_name}"
+        *) provider_name="2.dnscrypt-cert.${provider_name}" ;;
     esac
 
     [ -z "$ext_address" ] && usage
     case "$ext_address" in
         .*) usage ;;
-        0.*) echo "Do not use 0.0.0.0, use an actual external IP address" >&2 ; exit 1 ;;
+        0.*)
+            echo "Do not use 0.0.0.0, use an actual external IP address" >&2
+            exit 1
+            ;;
     esac
 
     echo "Provider name: [$provider_name]"
     cd "$KEYS_DIR"
     /opt/dnscrypt-wrapper/sbin/dnscrypt-wrapper \
         --gen-provider-keypair --nolog --dnssec --nofilter \
-        --provider-name="$provider_name" --ext-address="$ext_address" | \
+        --provider-name="$provider_name" --ext-address="$ext_address" |
         tee "${KEYS_DIR}/provider-info.txt"
     chmod 640 "${KEYS_DIR}/secret.key"
     chmod 644 "${KEYS_DIR}/public.key"
@@ -108,7 +111,10 @@ EOT
 
 case "$action" in
     start) start ;;
-    init) shift ; init $* ;;
+    init)
+        shift
+        init $*
+        ;;
     provider-info) provider_info ;;
     *) usage ;;
 esac

unbound-check.sh

@@ -1,4 +1,4 @@
 #! /usr/bin/env bash
 
-drill  -DQ -p 553 NS . @127.0.0.1 && \
+drill -DQ -p 553 NS . @127.0.0.1 &&
     drill -tDQ -p 553 NS . @127.0.0.1

unbound.sh

@@ -86,11 +86,11 @@ remote-control:
   control-interface: 127.0.0.1
 EOT
 
-mkdir -p /opt/unbound/etc/unbound/dev && \
+mkdir -p /opt/unbound/etc/unbound/dev &&
     cp -a /dev/random /dev/urandom /opt/unbound/etc/unbound/dev/
 
-mkdir -p -m 700 /opt/unbound/etc/unbound/var && \
-chown _unbound:_unbound /opt/unbound/etc/unbound/var && \
+mkdir -p -m 700 /opt/unbound/etc/unbound/var &&
+    chown _unbound:_unbound /opt/unbound/etc/unbound/var &&
     /opt/unbound/sbin/unbound-anchor -a /opt/unbound/etc/unbound/var/root.key
 
 if [ ! -f /opt/unbound/etc/unbound/unbound_control.pem ]; then

watchdog.sh

@@ -12,5 +12,5 @@ GRACE_PERIOD=60
 provider_key=$(cat "${KEYS_DIR}/public.key.txt")
 provider_name=$(cat "${KEYS_DIR}/provider_name")
 
-drill -p 443 -Q TXT "$provider_name" @127.0.0.1 || \
+drill -p 443 -Q TXT "$provider_name" @127.0.0.1 ||
     sv force-restart dnscrypt-wrapper