Merge pull request rapid7#20342 from cgranleese-r7/runs-trailing-comma-in-arguements-rubocop-on-modules

Runs Style/TrailingCommaInArguments Rubocop against modules

Author: adfoster-r7

modules/auxiliary/admin/oracle/oraenum.rb

@@ -71,7 +71,7 @@ def run
     ver.each do |v|
       print_status("\t#{v.chomp}")
       report_ora_enum_note(
-        { :component_version => v.chomp },
+        { :component_version => v.chomp }
       )
     end
 
@@ -85,24 +85,24 @@ def run
       if vparm['audit_trail'] == 'NONE'
         print_status("\tDatabase Auditing is not enabled!")
         report_ora_enum_note(
-          { :audit_trail => 'Disabled' },
+          { :audit_trail => 'Disabled' }
         )
       else
         print_status("\tDatabase Auditing is enabled!")
         report_ora_enum_note(
-          { :audit_trail => 'Enabled' },
+          { :audit_trail => 'Enabled' }
         )
       end
 
       if vparm['audit_sys_operations'] == 'FALSE'
         print_status("\tAuditing of SYS Operations is not enabled!")
         report_ora_enum_note(
-          { :audit_sys_ops => 'Disabled' },
+          { :audit_sys_ops => 'Disabled' }
         )
       else
         print_status("\tAuditing of SYS Operations is enabled!")
         report_ora_enum_note(
-          { :audit_sys_ops => 'Enabled' },
+          { :audit_sys_ops => 'Enabled' }
         )
       end
     end

modules/auxiliary/gather/ms14_052_xmldom.rb

@@ -31,7 +31,7 @@ def initialize(info = {})
         ],
         'Platform' => 'win',
         # MSB. Used in the wild since Feb 2014
-        'DisclosureDate' => '2014-09-09',
+        'DisclosureDate' => '2014-09-09'
       )
     )
 

modules/auxiliary/gather/windows_deployment_services_shares.rb

@@ -28,7 +28,7 @@ def initialize(info = {})
         'References' => [
           [ 'URL', 'http://technet.microsoft.com/en-us/library/cc749415(v=ws.10).aspx'],
           [ 'URL', 'http://rewtdance.blogspot.com/2012/11/windows-deployment-services-clear-text.html'],
-        ],
+        ]
       )
     )
 

modules/auxiliary/scanner/http/appletv_login.rb

@@ -64,7 +64,7 @@ def run_host(ip)
         blank_passwords: datastore['BLANK_PASSWORDS'],
         pass_file: datastore['PASS_FILE'],
         username: 'AirPlay',
-        user_as_pass: datastore['USER_AS_PASS'],
+        user_as_pass: datastore['USER_AS_PASS']
       )
       cred_collection = prepend_db_passwords(cred_collection)
     else
@@ -78,7 +78,7 @@ def run_host(ip)
         cred_details: cred_collection,
         stop_on_success: datastore['STOP_ON_SUCCESS'],
         bruteforce_speed: datastore['BRUTEFORCE_SPEED'],
-        connection_timeout: 5,
+        connection_timeout: 5
       )
     )
 

modules/auxiliary/scanner/http/es_file_explorer_open_port.rb

@@ -65,7 +65,7 @@ def http_post(command)
       'uri' => '/',
       'method' => 'POST',
       'data' => "{ \"command\":#{command} }",
-      'ctype' => 'application/json',
+      'ctype' => 'application/json'
     )
   end
 
@@ -254,7 +254,7 @@ def run_host(target_host)
       res = send_request_raw(
         'uri' => datastore['ACTIONITEM'],
         'method' => 'GET',
-        'ctype' => 'application/json',
+        'ctype' => 'application/json'
       )
       unless res
         print_error("#{peer}- Error Connecting")
@@ -274,7 +274,7 @@ def run_host(target_host)
         'uri' => '/',
         'method' => 'POST',
         'data' => "{ \"command\":appLaunch, \"appPackageName\":#{datastore['ACTIONITEM']} }",
-        'ctype' => 'application/json',
+        'ctype' => 'application/json'
       )
       unless res
         print_error("#{peer}- Error Connecting")

modules/auxiliary/scanner/http/frontpage_credential_dump.rb

@@ -27,7 +27,7 @@ def initialize(info = {})
           'Aditya K Sood @adityaksood', # Sparty tool'
           'Stephen Haywood @averagesecguy' # Metasploit module'
         ],
-        'License' => MSF_LICENSE,
+        'License' => MSF_LICENSE
       )
     )
 

modules/auxiliary/scanner/http/trace.rb

@@ -46,7 +46,7 @@ def run_host(target_host)
           :port => rport,
           :proto => 'tcp',
           :sname => (ssl ? 'https' : 'http'),
-          :info => "Vulnerable to Cross-Site Tracing",
+          :info => "Vulnerable to Cross-Site Tracing"
         )
       else
         vprint_error("#{rhost}:#{rport} returned #{res.code} #{res.message}")

modules/auxiliary/scanner/misc/cisco_smart_install.rb

@@ -33,7 +33,7 @@ def initialize(info = {})
         'Actions' => [
           ['SCAN', 'Description' => 'Scan for instances communicating via Smart Install Protocol (default)'],
           ['DOWNLOAD', 'Description' => 'Retrieve configuration via Smart Install Protocol']
-        ],
+        ]
       )
     )
 

modules/auxiliary/scanner/scada/moxa_discover.rb

@@ -76,15 +76,15 @@ def scanner_postscan(_batch)
       # Report the host
       report_host(
         :host => host,
-        :info => "Moxa Device",
+        :info => "Moxa Device"
       )
 
       # Report the service
       report_service(
         host: host,
         proto: 'udp',
         port: rport,
-        name: 'Moxa Protocol',
+        name: 'Moxa Protocol'
       )
 
       if response.empty?

modules/auxiliary/scanner/smb/smb_enumshares.rb

@@ -40,7 +40,7 @@ def initialize(info = {})
         'License' => MSF_LICENSE,
         'DefaultOptions' => {
           'DCERPC::fake_bind_multi' => false
-        },
+        }
       )
     )