From cdbba532998e5936bc2639bdcc6fda1952dea593 Mon Sep 17 00:00:00 2001
From: Tom Scavo <trscavo@internet2.edu>
Date: Tue, 2 May 2017 09:21:09 -0400
Subject: [PATCH] Trim redundant code

---
 lib/http_tools.sh | 36 +++++++++++++++---------------------
 1 file changed, 15 insertions(+), 21 deletions(-)

diff --git a/lib/http_tools.sh b/lib/http_tools.sh
index 6cf42bb..a343167 100755
--- a/lib/http_tools.sh
+++ b/lib/http_tools.sh
@@ -163,7 +163,7 @@ conditional_get () {
 		echo "ERROR: $FUNCNAME: directory does not exist: $tmp_dir" >&2
 		return 2
 	fi
-	$verbose_mode && print_log_message -D "$FUNCNAME using temporary directory $tmp_dir"
+	print_log_message -D "$FUNCNAME using temporary directory $tmp_dir"
 
 	# a cache directory is required
 	if [ -z "$cache_dir" ]; then
@@ -174,7 +174,7 @@ conditional_get () {
 		echo "ERROR: $FUNCNAME: directory does not exist: $cache_dir" >&2
 		return 2
 	fi
-	$verbose_mode && print_log_message -D "$FUNCNAME using cache directory $cache_dir"
+	print_log_message -D "$FUNCNAME using cache directory $cache_dir"
 
 	# determine the URL location
 	shift $(( OPTIND - 1 ))
@@ -187,7 +187,7 @@ conditional_get () {
 		echo "ERROR: $FUNCNAME: empty URL argument" >&2
 		return 2
 	fi
-	$verbose_mode && print_log_message -D "$FUNCNAME using location $location"
+	print_log_message -D "$FUNCNAME using location $location"
 	
 	#######################################################################
 	#
@@ -212,10 +212,8 @@ conditional_get () {
 	cached_header_file="$cache_dir/${hash}_headers"
 	cached_content_file="$cache_dir/${hash}_content"
 
-	if $verbose_mode; then
-		print_log_message -D "$FUNCNAME using cached header file: $cached_header_file"
-		print_log_message -D "$FUNCNAME using cached content file: $cached_content_file"
-	fi
+	print_log_message -D "$FUNCNAME using cached header file: $cached_header_file"
+	print_log_message -D "$FUNCNAME using cached content file: $cached_content_file"
 
 	# check if the resource is cached
 	if [ -f "$cached_header_file" ] && [ -f "$cached_content_file" ]; then
@@ -254,11 +252,9 @@ conditional_get () {
 	tmp_content_file="$tmp_dir/${FUNCNAME}_curl_content"
 	tmp_stderr_file="$tmp_dir/${FUNCNAME}_curl_stderr"
 
-	if $verbose_mode; then
-		print_log_message -D "$FUNCNAME using temp header file: ${tmp_header_file}"
-		print_log_message -D "$FUNCNAME using temp content file: ${tmp_content_file}"
-		print_log_message -D "$FUNCNAME using temp stderr file: ${tmp_stderr_file}"
-	fi
+	print_log_message -D "$FUNCNAME using temp header file: ${tmp_header_file}"
+	print_log_message -D "$FUNCNAME using temp content file: ${tmp_content_file}"
+	print_log_message -D "$FUNCNAME using temp stderr file: ${tmp_stderr_file}"
 
 	#######################################################################
 	#
@@ -311,7 +307,7 @@ conditional_get () {
 
 	# invoke curl
 	cmd="/usr/bin/curl $curl_opts $location"
-	$verbose_mode && print_log_message -D "$FUNCNAME issuing curl command: $cmd"
+	print_log_message -D "$FUNCNAME issuing curl command: $cmd"
 	eval $cmd
 	exit_code=$?
 	if [ $exit_code -ne 0 ]; then
@@ -364,13 +360,11 @@ conditional_get () {
 			print_log_message -W "$FUNCNAME: Content-Length response header missing"
 		fi
 
-		if $verbose_mode; then
-			print_log_message -D "$FUNCNAME downloaded ${actual_content_length} bytes"
-			if $do_conditional_get; then
-				print_log_message -D "$FUNCNAME refreshing cache files"
-			else
-				print_log_message -D "$FUNCNAME initializing cache files"
-			fi
+		print_log_message -D "$FUNCNAME downloaded ${actual_content_length} bytes"
+		if $do_conditional_get; then
+			print_log_message -D "$FUNCNAME refreshing cache files"
+		else
+			print_log_message -D "$FUNCNAME initializing cache files"
 		fi
 
 		# update the cache; maintain cache integrity at all times
@@ -397,7 +391,7 @@ conditional_get () {
 			return 1
 		fi
 		
-		$verbose_mode && print_log_message -D "$FUNCNAME downloaded 0 bytes (cache is up-to-date)"
+		print_log_message -D "$FUNCNAME downloaded 0 bytes (cache is up-to-date)"
 	else
 		print_log_message -E "$FUNCNAME failed with HTTP response code $response_code"
 		return 9