Updated code style as per @hmoore-r7's instructions

modules/auxiliary/scanner/http/title.rb

@@ -40,12 +40,18 @@ class Metasploit3 < Msf::Auxiliary
 
   def run_host(target_host)
     begin
+        # Send a normal GET request
         res = send_request_cgi('uri'          => '/',
                                'method'       => 'GET')
 
+        # If no response, quit now
         if res.nil?
-        print_error("No response from #{target_host}:#{rport}") if datastore['SHOW_ERRORS'] == true
+          print_error("[#{target_host}:#{rport}] No response") if datastore['SHOW_ERRORS'] == true
-      else
+          return
+        end
+
+        # Retrieve the headers to capture the Location and Server header
+        # Note that they are case-insensitive but stored in a hash
         server_header = nil
         location_header = nil
         if !res.headers.nil?
@@ -54,13 +60,24 @@ class Metasploit3 < Msf::Auxiliary
             server_header  = val if key.downcase == 'server'
           end
         else
-          print_error("No headers from #{target_host}:#{rport}") if datastore['SHOW_ERRORS'] == true
+          print_error("[#{target_host}:#{rport}] No HTTP headers") if datastore['SHOW_ERRORS'] == true
         end
 
-        if !res.body.nil?
+        # If the body is blank, just stop now as there is no chance of a title
-          # Very basic, just match the first title tag we come to.
+        if res.body.nil?
+          print_error("[#{target_host}:#{rport}] No webpage body") if datastore['SHOW_ERRORS'] == true
+          return
+        end
+
+        # Very basic, just match the first title tag we come to. If the match fails,
+        # there is no chance that we will have a title
         rx = %r{<title>[\n\t\s]*(?<title>.+?)[\s\n\t]*</title>}im.match(res.body.to_s)
-          if rx
+        unless rx
+          print_error("[#{target_host}:#{rport}] No webpage title") if datastore['SHOW_ERRORS'] == true
+          return
+        end
+
+        # Last bit of logic to capture the title
         rx[:title].strip!
         if rx[:title] != ''
           rx_title = CGI.unescapeHTML(rx[:title])
@@ -70,13 +87,7 @@ class Metasploit3 < Msf::Auxiliary
             report_note(host: target_host, type: "http.title", data: notedata)
           end
         else
-              print_error("No webpage title from #{target_host}:#{rport}") if datastore['SHOW_ERRORS'] == true
+          print_error("[#{target_host}:#{rport}] No webpage title") if datastore['SHOW_ERRORS'] == true
-            end
-          else
-            print_error("No webpage title from #{target_host}:#{rport}") if datastore['SHOW_ERRORS'] == true
-          end
-        else
-          print_error("No webpage body from #{target_host}:#{rport}") if datastore['SHOW_ERRORS'] == true
         end
       end
 
@@ -84,4 +95,3 @@ class Metasploit3 < Msf::Auxiliary
       rescue ::Timeout::Error, ::Errno::EPIPE
   end
 end
-end