ftp_version: Use FTP mixin

Author: g0tmi1k

modules/auxiliary/scanner/ftp/ftp_version.rb

@@ -6,7 +6,6 @@
 class MetasploitModule < Msf::Auxiliary
   include Msf::Exploit::Remote::Ftp
   include Msf::Auxiliary::Scanner
-  include Msf::Auxiliary::Report
 
   def initialize
     super(
@@ -29,33 +28,17 @@ def initialize
       ]
     )
 
-    # XXXX: One day, might be nice to enum via doing: `send_cmd(['xxx'])` {STAT,SYST,FEAT}
+    # TODO: One day, might be nice to enum via doing: `send_cmd(['xxx'])` {STAT,SYST,FEAT}
     deregister_options('FTPUSER', 'FTPPASS')
   end
 
-  def sanitize_ftp_response(str)
-    Rex::Text.to_hex_ascii(str.to_s.gsub(/^\d{3}[\s-]/, '').strip.gsub(/\A\(|\)\z/, ''))
-  end
-
   def run_host(_target_host)
     connect(true, false)
 
     if banner
-      banner_sanitized = sanitize_ftp_response(banner)
-      print_good("FTP Banner: #{banner_sanitized}")
-      report_service(
-        host: rhost,
-        port: rport,
-        name: 'ftp',
-        info: banner_sanitized
-      )
-      report_note(
-        host: rhost,
-        port: rport,
-        proto: 'tcp',
-        type: 'ftp.banner',
-        data: { banner: banner.strip }
-      )
+      print_good("FTP Banner: #{Rex::Text.to_hex_ascii(banner_version)}")
+    else
+      vprint_warning('No FTP banner received')
     end
   rescue ::Rex::ConnectionRefused
     vprint_error('Connection refused')