Use -n rather than -z for the endpoint var

commands/exists.sh

@@ -10,7 +10,7 @@ fi
 echo "Checking for file existence at s3://${S3_BUCKET}/${FILE} at the ${S3_ENDPOINT}"
 
 # Verify file existence.
-sh -c "aws s3api head-object --bucket ${S3_BUCKET} --key ${FILE} $S3_PROFILE $ENDPOINT_APPEND $*"
+sh -c "aws s3api head-object --bucket ${S3_BUCKET} --key ${FILE} ${S3_PROFILE} ${ENDPOINT_APPEND} $*"
 
 # XXX: we are just checking the error code, but should check the result for a 404, and raise error in other cases
 if [ $? == 0 ]

commands/ls.sh

@@ -10,7 +10,7 @@ fi
 echo $ENDPOINT_APPEND
 
 # Verify file existence.
-output=$(sh -c "aws s3 ls s3://${S3_BUCKET}/${FILE} $S3_PROFILE $ENDPOINT_APPEND $*")
+output=$(sh -c "aws s3 ls s3://${S3_BUCKET}/${FILE} ${S3_PROFILE} ${ENDPOINT_APPEND} $*")
 
 echo $output
 

commands/rm.sh

@@ -8,7 +8,7 @@ if [ -z "$FILE" ]; then
 fi
 
 # Verify file existence.
-output=$(sh -c "aws s3 rm s3://${S3_BUCKET}/${FILE} $S3_PROFILE $ENDPOINT_APPEND $*")
+output=$(sh -c "aws s3 rm s3://${S3_BUCKET}/${FILE} ${S3_PROFILE} ${ENDPOINT_APPEND} $*")
 
 echo $output
 

commands/sync.sh

@@ -3,4 +3,4 @@
 set -e
 
 # Sync path.
-sh -c "aws s3 sync ${SOURCE_DIR:-.} s3://${S3_BUCKET}/${DEST_DIR} $S3_PROFILE --no-progress $ENDPOINT_APPEND $*"
+sh -c "aws s3 sync ${SOURCE_DIR:-.} s3://${S3_BUCKET}/${DEST_DIR} ${S3_PROFILE} --no-progress ${ENDPOINT_APPEND} $*"

entrypoint.sh

@@ -24,15 +24,15 @@ fi
 
 # Default to us-east-1 if AWS_REGION not set.
 if [ -z "$S3_REGION" ]; then
-  export S3_REGION="us-east-1"
+  S3_REGION="us-east-1"
 fi
 
 # Override default AWS endpoint if user sets S3_ENDPOINT
-if [ -z "$S3_ENDPOINT" ]; then
-  export ENDPOINT_APPEND="--endpoint-url ${S3_ENDPOINT}"
+if [ -n "$S3_ENDPOINT" ]; then
+  ENDPOINT_APPEND="--endpoint-url ${S3_ENDPOINT}"
 fi
 
-export S3_PROFILE="--profile action-s3-utility"
+S3_PROFILE="--profile action-s3-utility"
 
 # Create a dedicated profile for this action
 aws configure --profile action-s3-utility <<-EOF > /dev/null 2>&1