Closes byrnedo#9: conditional binary operator expected

README.md

@@ -1,4 +1,5 @@
 # Docker Registry Tool
+[![Build Status](https://travis-ci.org/vladkras/docker-reg-tool.svg?branch=master)](https://travis-ci.org/vladkras/docker-reg-tool)
 
 Small bash script to do repetitive things with docker registry > 2.1 ( I think, whichever version supports delete by manifest)
 
@@ -8,10 +9,10 @@ Small bash script to do repetitive things with docker registry > 2.1 ( I think,
 [Jq](https://stedolan.github.io/jq/)
 
 ```
-Usage: 
+Usage:
 
     ./docker_reg_tool REGISTRY_BASE_URL ACTION [OPTIONS..]
-    
+
     Actions:
 
     - list               list repos
@@ -38,7 +39,7 @@ Usage:
 ...are sniffed out of ~/.docker/config.json. So you need to do `docker login...` before you can use this tool.
 
 They can also be set with the `BASIC_AUTH` environment variable.
-    
+
     BASIC_AUTH=user:pass ./docker_reg_tool ...
 
 Only been tested on Basic auth.
@@ -58,6 +59,3 @@ Set the `TRACE` environment variable to true to turn on `set -x`
 ## Docker Image
 
 See https://hub.docker.com/r/byrnedo/reg-tool/
-
-
-

docker_reg_tool

@@ -1,5 +1,5 @@
 #!/bin/bash
-set -euo pipefail
+set -eo pipefail
 
 CMD=$0
 
@@ -12,18 +12,18 @@ function isTruthy {
 
 }
 
-if [[ -v "TRACE" ]] && [[ $(isTruthy "$TRACE") ]]; then
+if [[ ${TRACE} ]] && [[ $(isTruthy "$TRACE") ]]; then
     set -x
 fi
 
 
 function usage {
     cat <<EOU
 
-    Usage: 
+    Usage:
 
     $CMD REGISTRY_BASE_URL ACTION [OPTIONS..]
-    
+
     Actions:
 
     - list               list repos
@@ -67,7 +67,7 @@ shift
 CREDS=""
 DOCKER_CONFIG="$HOME/.docker/config.json"
 
-if [[ -v "BASIC_AUTH" ]]; then
+if [[ ${BASIC_AUTH} ]]; then
     CREDS="Authorization: Basic $(echo -n $BASIC_AUTH|base64)"
 elif [[ -f "$DOCKER_CONFIG" ]]; then
     AUTH_INFO=$(jq -r '.auths["'$REG'"].auth' < "$DOCKER_CONFIG")
@@ -83,7 +83,7 @@ fi
 
 SEC_FLAG=""
 
-if [[ -v "INSECURE_REGISTRY" ]] && [[ $(isTruthy "$INSECURE_REGISTRY") ]]; then
+if [[ ${INSECURE_REGISTRY} ]] && [[ $(isTruthy "$INSECURE_REGISTRY") ]]; then
     SEC_FLAG="-k"
 fi
 
@@ -116,10 +116,9 @@ case "$ACTION" in
         if [ $result -eq 202 ]; then
             echo "Successfully deleted"
             exit 0
-        else 
+        else
             echo "Failed to delete"
             exit 3
         fi
         ;;
 esac
-