Merge branch 'only-alert-on-both-dns-errors' into 'master'

Alert in more detail on DNS failures.

See merge request !10

1 files changed, 2 insertions, 2 deletions

diff --git a/lib/custodian/protocoltest/http.rb b/lib/custodian/protocoltest/http.rb
index 67b4660..a3f34cc 100644
--- a/lib/custodian/protocoltest/http.rb
+++ b/lib/custodian/protocoltest/http.rb
@@ -338,7 +338,7 @@ module Custodian
             errors << "#{protocol_msg}: More than 10 redirections."
           rescue Curl::Err::HostResolutionError => x
             # Log the DNS error-message.
-            resolution_errors << "#{resolve_mode} - #{x.message}"
+            resolution_errors << "#{protocol_msg}: #{x.class}: #{x.message}\n  #{x.backtrace.join("\n  ")}."
           rescue => x
             errors << "#{protocol_msg}: #{x.class}: #{x.message}\n  #{x.backtrace.join("\n  ")}."
           end
@@ -359,7 +359,7 @@ module Custodian
         end
 
         # uh-oh! Resolution failed on both protocols!
-        if resolution_errors.length > 0
+        if resolution_errors.length > 1
           errors << "DNS Error when resolving host - #{resolution_errors.join(',')}"
         end