Merge pull request #20330 from cgranleese-r7/fixes-conditional

Fixes issues in a few modules

Author: adfoster-r7

modules/auxiliary/gather/zabbix_toggleids_sqli.rb

@@ -239,7 +239,7 @@ def authenticate
        end
 
        sid = match[1]
-     elsif
+     else
        fail_with(Failure::Unknown, 'Server did not respond in an expected way')
      end
    end

modules/auxiliary/scanner/http/cert.rb

@@ -65,7 +65,7 @@ def run_host(ip)
 
     if ( a < 1 or b > 0 )
       print_good("#{ip} - '#{vhostn}' : '" + before.to_s + "' - '" + after.to_s + "' (EXPIRED)'")
-    elsif
+    else
       # show verbose as status
       print_status("#{ip} - '#{vhostn}' : '" + before.to_s + "' - '" + after.to_s + "'")
     end

modules/auxiliary/scanner/lotus/lotus_domino_version.rb

@@ -77,12 +77,8 @@ def run_host(ip)
           else
             ''
           end
-        elsif
-          if (res.code and res.headers['Location'])
-            print_error("#{ip}:#{rport} #{res.code} Redirect to #{res.headers['Location']}")
-          else
-            ''
-          end
+        elsif (res.code and res.headers['Location'])
+          print_error("#{ip}:#{rport} #{res.code} Redirect to #{res.headers['Location']}")
         else
           ''
         end

modules/exploits/windows/http/cogent_datahub_command.rb

@@ -49,10 +49,8 @@ module has been tested successfully with Cogent DataHub 7.3.4 on
     register_options(
       [
         OptString.new('URIPATH',   [ true,  'The URI to use (do not change)', '/']),
-        OptPort.new('SRVPORT',     [ true,  'The daemon port to listen on ' +
-                                                      '(do not change)', 80 ]),
-        OptInt.new('WEBDAV_DELAY', [ true,  'Time that the HTTP Server will ' +
-                                          'wait for the payload request', 20]),
+        OptPort.new('SRVPORT',     [ true,  'The daemon port to listen on (do not change)', 80 ]),
+        OptInt.new('WEBDAV_DELAY', [ true,  'Time that the HTTP Server will wait for the payload request', 20]),
         OptString.new('UNCPATH',   [ false, 'Override the UNC path to use.' ])
       ])
   end