oxidized 0.33.0 → 0.34.0

data/lib/oxidized/input/ssh.rb

@@ -16,7 +16,7 @@ module Oxidized
     include Input::CLI
     class NoShell < OxidizedError; end
 
-    def connect(node)
+    def connect(node) # rubocop:disable Naming/PredicateMethod
       @node        = node
       @output      = String.new('')
       @pty_options = { term: "vt100" }
@@ -24,10 +24,10 @@ module Oxidized
       if Oxidized.config.input.debug?
         logfile = Oxidized::Config::LOG + "/#{@node.ip}-ssh"
         @log = File.open(logfile, 'w')
-        Oxidized.logger.debug "lib/oxidized/input/ssh.rb: I/O Debuging to #{logfile}"
+        logger.debug "I/O Debuging to #{logfile}"
       end
 
-      Oxidized.logger.debug "lib/oxidized/input/ssh.rb: Connecting to #{@node.name}"
+      logger.debug "Connecting to #{@node.name}"
       @ssh = Net::SSH.start(@node.ip, @node.auth[:username], make_ssh_opts)
       unless @exec
         shell_open @ssh
@@ -45,9 +45,9 @@ module Oxidized
     end
 
     def cmd(cmd, expect = node.prompt)
-      Oxidized.logger.debug "lib/oxidized/input/ssh.rb #{cmd.dump} @ #{node.name} with expect: #{expect.inspect}"
+      logger.debug "Sending '#{cmd.dump}' @ #{node.name} with expect: #{expect.inspect}"
       if Oxidized.config.input.debug?
-        @log.puts "sent #{cmd.dump}"
+        @log.puts "sent cmd #{@exec ? cmd.dump : (cmd + newline).dump}"
         @log.flush
       end
       cmd_output = if @exec
@@ -60,6 +60,10 @@ module Oxidized
     end
 
     def send(data)
+      if Oxidized.config.input.debug?
+        @log.puts "sent data #{data.dump}"
+        @log.flush
+      end
       @ses.send_data data
     end
 
@@ -75,8 +79,11 @@ module Oxidized
       disconnect_cli
       # if disconnect does not disconnect us, give up after timeout
       Timeout.timeout(Oxidized.config.timeout) { @ssh.loop }
-    rescue Errno::ECONNRESET, Net::SSH::Disconnect, IOError
-      # These exceptions are intented and therefore not handled here
+    rescue Errno::ECONNRESET, Net::SSH::Disconnect, IOError => e
+      logger.debug 'The other side closed the connection while ' \
+                   "disconnecting, rasing #{e.class} with #{e.messages}"
+    rescue Timeout::Error
+      logger.debug "#{@node.name} timed out while disconnecting"
     ensure
       @log.close if Oxidized.config.input.debug?
       (@ssh.close rescue true) unless @ssh.closed?
@@ -110,7 +117,7 @@ module Oxidized
 
     def cmd_shell(cmd, expect_re)
       @output = String.new('')
-      @ses.send_data cmd + "\n"
+      @ses.send_data cmd + newline
       @ses.process
       expect expect_re if expect_re
       @output
@@ -118,7 +125,7 @@ module Oxidized
 
     def expect(*regexps)
       regexps = [regexps].flatten
-      Oxidized.logger.debug "lib/oxidized/input/ssh.rb: expecting #{regexps.inspect} at #{node.name}"
+      logger.debug "Expecting #{regexps.inspect} at #{node.name}"
       Timeout.timeout(Oxidized.config.timeout) do
         @ssh.loop(0.1) do
           sleep 0.1
@@ -145,7 +152,7 @@ module Oxidized
 
       auth_methods = vars(:auth_methods) || %w[none publickey password]
       ssh_opts[:auth_methods] = auth_methods
-      Oxidized.logger.debug "AUTH METHODS::#{auth_methods}"
+      logger.debug "AUTH METHODS::#{auth_methods}"
 
       ssh_opts[:proxy] = make_ssh_proxy_command(vars(:ssh_proxy), vars(:ssh_proxy_port), secure) if vars(:ssh_proxy)
 
@@ -155,10 +162,10 @@ module Oxidized
       ssh_opts[:host_key]   = vars(:ssh_host_key).split(/,\s*/)   if vars(:ssh_host_key)
       ssh_opts[:hmac]       = vars(:ssh_hmac).split(/,\s*/)       if vars(:ssh_hmac)
 
-      if Oxidized.config.input.debug?
-        ssh_opts[:logger]  = Oxidized.logger
-        ssh_opts[:verbose] = Logger::DEBUG
-      end
+      # Use our logger for Net:SSH
+      ssh_logger = SemanticLogger[Net::SSH]
+      ssh_logger.level = Oxidized.config.input.debug? ? :debug : :error
+      ssh_opts[:logger] = ssh_logger
 
       ssh_opts
     end

data/lib/oxidized/input/telnet.rb

@@ -6,7 +6,7 @@ module Oxidized
     include Input::CLI
     attr_reader :telnet
 
-    def connect(node)
+    def connect(node) # rubocop:disable Naming/PredicateMethod
       @node    = node
       @timeout = Oxidized.config.timeout
       @node.model.cfg['telnet'].each { |cb| instance_exec(&cb) }
@@ -35,7 +35,7 @@ module Oxidized
     end
 
     def cmd(cmd_str, expect = @node.prompt)
-      Oxidized.logger.debug "Telnet: #{cmd_str} @#{@node.name}"
+      logger.debug "Telnet: #{cmd_str} @#{@node.name}"
       return send(cmd_str + "\r\n") unless expect
 
       # create a string to be passed to oxidized_expect and modified _there_
@@ -78,7 +78,7 @@ module Net
     ## FIXME: we also need output (not sure I'm going to support this)
     attr_reader :output
 
-    def oxidized_expect(options) ## rubocop:disable Metrics/PerceivedComplexity
+    def oxidized_expect(options)
       model    = @options["Model"]
       @log     = @options["Log"]
 

data/lib/oxidized/input/tftp.rb

@@ -21,7 +21,7 @@ module Oxidized
     end
 
     def cmd(file)
-      Oxidized.logger.debug "TFTP: #{file} @ #{@node.name}"
+      logger.debug "TFTP: #{file} @ #{@node.name}"
       config = StringIO.new
       @tftp.getbinary file, config
       config.rewind

data/lib/oxidized/job.rb

@@ -1,16 +1,19 @@
 module Oxidized
   class Job < Thread
+    include SemanticLogger::Loggable
+
     attr_reader :start, :end, :status, :time, :node, :config
 
     def initialize(node)
-      @node         = node
-      @start        = Time.now.utc
+      @node = node
+      @start = Time.now.utc
+      self.name = "Oxidized::Job '#{@node.name}'"
       super do
-        Oxidized.logger.debug "lib/oxidized/job.rb: Starting fetching process for #{@node.name} at #{Time.now.utc}"
+        logger.debug "Starting fetching process for #{@node.name} at #{Time.now.utc}"
         @status, @config = @node.run
         @end             = Time.now.utc
         @time            = @end - @start
-        Oxidized.logger.debug "lib/oxidized/job.rb: Config fetched for #{@node.name} at #{@end}"
+        logger.debug "Config fetched for #{@node.name} at #{@end}"
       end
     end
   end

data/lib/oxidized/logger.rb

@@ -0,0 +1,51 @@
+require 'semantic_logger'
+
+module Oxidized
+  module Logger
+    include SemanticLogger::Loggable
+
+    def self.setup
+      config = Oxidized.config
+      FileUtils.mkdir_p(Config::LOG) unless File.directory?(Config::LOG)
+
+      SemanticLogger.add_signal_handler
+
+      if config.use_syslog?
+        SemanticLogger.add_appender(appender: :syslog)
+        logger.warn("The configuration 'use_syslog' is deprecated. " \
+                    "Remove it and use 'logger' instead")
+      elsif config.log?
+        SemanticLogger.add_appender(file_name: File.expand_path(config.log))
+        logger.warn("The configuration 'log' is deprecated. " \
+                    "Remove it and use 'logger' instead")
+      elsif config.logger?
+        SemanticLogger.default_level = config.logger.level if config.logger.level?
+        config.logger.appenders.each { |a| add_appender a } if config.logger.has_key?('appenders')
+      end
+
+      # No appenders configured
+      SemanticLogger.add_appender(io: $stderr) if SemanticLogger.appenders.empty?
+
+      return if %i[trace debug].include?(SemanticLogger.default_level)
+
+      SemanticLogger.default_level = :debug if config.debug?
+    end
+
+    def self.add_appender(appender)
+      case appender['type']
+      when 'file'
+        params = { file_name: File.expand_path(appender['file']) }
+      when 'stderr'
+        params = { io: $stderr }
+      when 'stdout'
+        params = { io: $stdout }
+      when 'syslog'
+        params = { appender: :syslog, application: "oxidized" }
+      else
+        raise InvalidConfig, "Unknown logger #{appender['type']}, edit #{Oxidized::Config.configfile}"
+      end
+      params[:level] = appender['level'] if appender.has_key?('level')
+      SemanticLogger.add_appender(**params)
+    end
+  end
+end

data/lib/oxidized/model/acos.rb

@@ -20,6 +20,7 @@ class ACOS < Oxidized::Model
     cfg.gsub! /\s(Memory).*/, ' \\1 <removed>'
     cfg.gsub! /\s(Current time is).*/, ' \\1 <removed>'
     cfg.gsub! /\s(The system has been up).*/, ' \\1 <removed>'
+    cfg.gsub! /\s(Hardware: \d+ CPUs\(Stepping \d+\). Single \d+G drive. Free storage is).*/, ' \\1 <removed>'
     comment cfg
   end
 

data/lib/oxidized/model/aos7.rb

@@ -33,10 +33,16 @@ class AOS7 < Oxidized::Model
   end
 
   cmd 'show running-directory' do |cfg|
+    # Remove extra lines occuring when the command runs slow
+    cfg.gsub! /^Please wait...\n/, ''
+    cfg.gsub! /^\n\n/, "\n"
     comment cfg
   end
 
   cmd 'show configuration snapshot' do |cfg|
+    # Remove extra lines occuring when the command runs slow
+    cfg.gsub! /^Please wait...\n/, ''
+    cfg.gsub! /^\n\n/, "\n"
     cfg
   end
 

data/lib/oxidized/model/aoscx.rb

@@ -58,6 +58,8 @@ class Aoscx < Oxidized::Model
     cfg.gsub! /^(\d\/\d\/\d.*\s+)\d+\s+$/, '\\1<hidden>'
     cfg.gsub! /^(\d+\/?\S+\s+\S+\s+)\d+\.\d+\s+C\s+(.*)/, '\\1<hidden>   \\2'
     cfg.gsub! /^(LC.*\s+)\d+\.\d+\s+(C.*)$/, '\\1 <hidden> \\2'
+    # match show environment No speed shown for switches CX83xx, e.g. "PSU-1/1/1      N/A      N/A            N/A     front-to-back  ok      7360"
+    cfg.gsub! /^PSU(\S+\s+\S+\s+\s+\S+\s+)(slow|normal|medium|fast|max|N\/A)\s+(\S+\s+\S+\s+)\d+[[:blank:]]+/, '\\1<speed> \\3<rpm>'
     cfg.gsub! /^(\S+\s+\S+\s+\s+\S+\s+)(slow|normal|medium|fast|max)\s+(\S+\s+\S+\s+)\d+[[:blank:]]+/, '\\1<speed> \\3<rpm>'
     # match show environment power-consumption on VSF or standadlone, non-chassis and non-6400 switch, e.g. "2    6300M 48G 4SFP56 Swch                 156.00      155.94"
     cfg.gsub! /^(\d+\s+.+\s+)(\s{2}\d{2}\.\d{2}|\s{1}\d{3}\.\d{2}|\d{4}\.\d{2})(\s+)(\s{2}\d{2}\.\d{2}|\s{1}\d{3}\.\d{2}|\d{4}\.\d{2})$/, '\\1<power>\\3<power>'

data/lib/oxidized/model/aosw.rb

@@ -25,24 +25,25 @@ class AOSW < Oxidized::Model
   end
 
   cmd :secret do |cfg|
-    cfg.gsub!(/secret (\S+)$/, 'secret <secret removed>')
-    cfg.gsub!(/enable secret (\S+)$/, 'enable secret <secret removed>')
-    cfg.gsub!(/PRE-SHARE (\S+)$/, 'PRE-SHARE <secret removed>')
-    cfg.gsub!(/ipsec (\S+)$/, 'ipsec <secret removed>')
-    cfg.gsub!(/community (\S+)$/, 'community <secret removed>')
+    cfg.gsub!(/secret (\S+)\s?$/, 'secret <secret removed>')
+    cfg.gsub!(/enable secret (\S+)\s?$/, 'enable secret <secret removed>')
+    cfg.gsub!(/PRE-SHARE (\S+)\s?$/, 'PRE-SHARE <secret removed>')
+    cfg.gsub!(/ipsec (\S+)\s?$/, 'ipsec <secret removed>')
+    cfg.gsub!(/community (\S+)\s?$/, 'community <secret removed>')
     cfg.gsub!(/ sha (\S+)/, ' sha <secret removed>')
     cfg.gsub!(/ des (\S+)/, ' des <secret removed>')
     cfg.gsub!(/mobility-manager (\S+) user (\S+) (\S+)/, 'mobility-manager \1 user \2 <secret removed>')
-    cfg.gsub!(/mgmt-user (\S+) (root|guest-provisioning|network-operations|read-only|location-api-mgmt) (\S+)$/, 'mgmt-user \1 \2 <secret removed>') # MAS & Wireless Controler
-    cfg.gsub!(/mgmt-user (\S+) (\S+)( (read-only|guest-mgmt))?$/, 'mgmt-user \1 <secret removed> \3') # IAP
+    cfg.gsub!(/mgmt-user (\S+) (root|guest-provisioning|network-operations|read-only|location-api-mgmt) (\S+)\s?$/, 'mgmt-user \1 \2 <secret removed>') # MAS & Wireless Controler
+    cfg.gsub!(/mgmt-user (\S+) (\S+)( (read-only|guest-mgmt))?\s?$/, 'mgmt-user \1 <secret removed> \3') # IAP
     # MAS format: mgmt-user <username> <accesslevel> <password hash>
     # IAP format (root user): mgmt-user <username> <password hash>
     # IAP format: mgmt-user <username> <password hash> <access level>
-    cfg.gsub!(/key (\S+)$/, 'key <secret removed>')
-    cfg.gsub!(/wpa-passphrase (\S+)$/, 'wpa-passphrase <secret removed>')
-    cfg.gsub!(/bkup-passwords (\S+)$/, 'bkup-passwords <secret removed>')
-    cfg.gsub!(/user (\S+) (\S+) (\S+)$/, 'user \1 <secret removed> \3')
-    cfg.gsub!(/virtual-controller-key (\S+)$/, 'virtual-controller-key <secret removed>')
+    cfg.gsub!(/key (\S+)\s?$/, 'key <secret removed>')
+    cfg.gsub!(/vrrp-passphrase (\S+)\s?$/, 'vrrp-passphrase <secret removed>')
+    cfg.gsub!(/wpa-passphrase (\S+)\s?$/, 'wpa-passphrase <secret removed>')
+    cfg.gsub!(/bkup-passwords (\S+)\s?$/, 'bkup-passwords <secret removed>')
+    cfg.gsub!(/ap-console-password (\S+)\s?$/, 'ap-console-password <secret removed>')
+    cfg.gsub!(/virtual-controller-key (\S+)\s?$/, 'virtual-controller-key <secret removed>')
     cfg
   end
 
@@ -52,22 +53,26 @@ class AOSW < Oxidized::Model
   end
 
   cmd 'show inventory' do |cfg|
-    cfg = "" if cfg =~ /(Invalid input detected at '\^' marker|Parse error)/ # Don't show for unsupported devices (IAP and MAS)
+    # Don't show for unsupported devices (IAP and MAS)
+    cfg = "" if cfg =~ /(Invalid input detected at '\^' marker|Parse error)/
     rstrip_cfg clean cfg
   end
 
   cmd 'show slots' do |cfg|
-    cfg = "" if cfg =~ /(Invalid input detected at '\^' marker|Parse error)/ # Don't show for unsupported devices (IAP and MAS)
+    # Don't show for unsupported devices (IAP and MAS)
+    cfg = "" if cfg =~ /(Invalid input detected at '\^' marker|Parse error)/
     rstrip_cfg comment cfg
   end
 
   cmd 'show license' do |cfg|
-    cfg = "" if cfg =~ /(Invalid input detected at '\^' marker|Parse error)/ # Don't show for unsupported devices (IAP and MAS)
+    # Don't show for unsupported devices (IAP and MAS)
+    cfg = "" if cfg =~ /(Invalid input detected at '\^' marker|Parse error)/
     rstrip_cfg comment cfg
   end
 
   cmd 'show license passphrase' do |cfg|
-    cfg = "" if cfg.match /(Invalid input detected at '\^' marker|Parse error)/ # Don't show for unsupported devices (IAP and MAS)
+    # Don't show for unsupported devices (IAP and MAS)
+    cfg = "" if cfg.match /(Invalid input detected at '\^' marker|Parse error)/
     rstrip_cfg comment cfg
   end
 
@@ -117,7 +122,7 @@ class AOSW < Oxidized::Model
       next if line =~ /Output \d Config/i
       next if line =~ /(Tachometers|Temperatures|Voltages)/
       next if line =~ /((Card|CPU) Temperature|Chassis Fan|VMON1[0-9])/
-      next if line =~ /[0-9]+\s+(RPMS?|m?V|C|W)/i
+      next if line =~ /[0-9.]{1,6}\s+(RPMS?|m?V|C|W)/i
 
       out << line.strip
     end

data/lib/oxidized/model/aricentiss.rb

@@ -37,7 +37,7 @@ class AricentISS < Oxidized::Model
   cmd 'show system information' do |cfg|
     cfg.sub! /^Device Up Time.*\n/, ''
     cfg.delete! "\r"
-    comment(cfg).gsub(/ +$/, '')
+    comment(cfg).rstrip
   end
 
   cmd 'show running-config' do |cfg|
@@ -53,6 +53,6 @@ class AricentISS < Oxidized::Model
       end
 
       l
-    end.join.gsub(/ +$/, '')
+    end.join.rstrip
   end
 end

data/lib/oxidized/model/asa.rb

@@ -16,8 +16,7 @@ class ASA < Oxidized::Model
     cfg.gsub! /^passwd (\S+) (.*)/, 'passwd <secret hidden> \2'
     cfg.gsub! /username (\S+) password (\S+) (.*)/, 'username \1 password <secret hidden> \3'
     cfg.gsub! /(ikev[12] ((remote|local)-authentication )?pre-shared-key( hex)?) (\S+)/, '\1 <secret hidden>'
-    cfg.gsub! /^(aaa-server TACACS\+? \(\S+\) host[^\n]*\n(\s+[^\n]+\n)*\skey) \S+$/mi, '\1 <secret hidden>'
-    cfg.gsub! /^(aaa-server \S+ \(\S+\) host[^\n]*\n(\s+[^\n]+\n)*\s+key) \S+$/mi, '\1 <secret hidden>'
+    cfg.gsub! /^(aaa-server \S+(?: \(\S+\))? host \S+\n(?: [^\n]+\n)* +key) \S+$/mi, '\1 <secret hidden>'
     cfg.gsub! /ldap-login-password (\S+)/, 'ldap-login-password <secret hidden>'
     cfg.gsub! /^snmp-server host (.*) community (\S+)/, 'snmp-server host \1 community <secret hidden>'
     cfg.gsub! /^(failover key) .+/, '\1 <secret hidden>'
@@ -97,7 +96,8 @@ class ASA < Oxidized::Model
         contexts = systemcfg.scan(/^context (\S+)$/)
         files = systemcfg.scan(/config-url (\S+)$/)
         contexts.each_with_index do |cont, i|
-          allcfg = allcfg + "\n\n----------========== [ CONTEXT " + cont.join(" ") + " FILE " + files[i].join(" ") + " ] ==========----------\n\n"
+          allcfg = allcfg + "\n\n----------========== [ CONTEXT " + cont.join(" ") +
+                   " FILE " + files[i].join(" ") + " ] ==========----------\n\n"
           cmd "more " + files[i].join(" ") do |cfgcontext|
             allcfg = allcfg + "\n\n" + cfgcontext
           end

data/lib/oxidized/model/awplus.rb

@@ -61,25 +61,28 @@ class AWPlus < Oxidized::Model
     cfg
   end
 
-  # Config required for telnet to detect username prompt
+  # Config required for telnet to detect username & password prompt.
   cfg :telnet do
     username /login:\s/
+    password /^Password:\s/
   end
 
-  # Main login config
+  # Config required for ssh to specify newline characters.
   cfg :telnet, :ssh do
+    newline "\r\n"
+
     post_login do
-      if vars :enable
-        send "enable\n"
-        expect /^Password:\s/
-        cmd vars(:enable) + "\r\n"
-      else
-        cmd 'enable' # Required for Priv-Exec users without enable PW to be put into "enable mode".
+      if vars(:enable) == true
+        cmd "enable"
+      elsif vars(:enable)
+        cmd "enable", /^[pP]assword:/
+        cmd vars(:enable)
       end
-      #      cmd 'terminal length 0' #set so the entire config is output without intervention.
+      # cmd 'terminal length 0' # set so the entire config is output without intervention.
     end
+
     pre_logout do
-      #      cmd 'terminal no length' #Sets term length back to default on exit.
+      # cmd 'terminal no length' # sets term length back to default on exit.
       send "exit\r\n"
     end
   end

data/lib/oxidized/model/edgecos.rb

@@ -32,7 +32,8 @@ class EdgeCOS < Oxidized::Model
   cmd 'show system' do |cfg|
     cfg.gsub! /^.*\sUp Time\s*:.*\n/i, ''
     cfg.gsub! /(\sTemperature \d*:)\s*\d+ degrees/, '\\1 <temperature values hidden>'
-    cfg.gsub! /^!?\s*Fan \d+ speed:\s+\d+ rpm\s+Fan \d+ speed:\s+\d+ rpm\s+Fan \d+ speed:\s+\d+ rpm$/, '<fan speeds hidden>'
+    cfg.gsub! /^!?\s*Fan \d+ speed:\s+\d+ rpm\s+Fan \d+ speed:\s+\d+ rpm\s+Fan \d+ speed:\s+\d+ rpm$/,
+              '<fan speeds hidden>'
     comment cfg
   end
 

data/lib/oxidized/model/edgeos.rb

@@ -1,7 +1,7 @@
 class Edgeos < Oxidized::Model
   using Refinements
 
-  # EdgeOS #
+  # Ubiquiti EdgeOS #
 
   prompt /@.*?:~\$\s/
 
@@ -10,11 +10,12 @@ class Edgeos < Oxidized::Model
   end
 
   cmd :secret do |cfg|
-    cfg.gsub! /encrypted-password (\S+).*/, 'encrypted-password <secret removed>'
-    cfg.gsub! /plaintext-password (\S+).*/, 'plaintext-password <secret removed>'
-    cfg.gsub! /password (\S+).*/, 'password <secret removed>'
-    cfg.gsub! /pre-shared-secret (\S+).*/, 'pre-shared-secret <secret removed>'
-    cfg.gsub! /community (\S+) {/, 'community <hidden> {'
+    cfg.gsub!(/(encrypted-password) \S+/, '\1 <secret removed>')
+    cfg.gsub!(/(plaintext-password) \S+/, '\1 <secret removed>')
+    cfg.gsub!(/(password) \S+/, '\1 <secret removed>')
+    cfg.gsub!(/(pre-shared-secret) \S+/, '\1 <secret removed>')
+    cfg.gsub!(/(community) \S+ {/, '\1 <hidden> {')
+    cfg.gsub!(/(commit-archive location) \S+/, '\1 <secret removed>')
     cfg
   end
 

data/lib/oxidized/model/edgeswitch.rb

@@ -8,7 +8,9 @@ class EdgeSwitch < Oxidized::Model
   prompt /\(.*\)\s[#>]/
 
   cmd 'show running-config' do |cfg|
-    cfg.each_line.to_a[2..-2].reject { |line| line.match(/System Up Time.*/) || line.match(/Current SNTP Synchronized Time.*/) }.join
+    cfg.each_line.to_a[2..-2].reject do |line|
+      line.match(/System Up Time.*/) || line.match(/Current SNTP Synchronized Time.*/)
+    end.join
   end
 
   cfg :telnet do

data/lib/oxidized/model/efos.rb

@@ -0,0 +1,41 @@
+class EFOS < Oxidized::Model
+  using Refinements
+
+  # Enhanced Fabric OS - Broadcom
+  comment '! '
+  prompt /^([\w.@()-]+[#>]\s?)$/
+
+  cmd :all do |cfg|
+    # Remove the echo of the entered command and the prompt after it
+    cfg.cut_both
+  end
+
+  cmd 'show bootvar' do |cfg|
+    comment cfg
+  end
+
+  cmd 'show fiber-ports optical-transceiver-info all' do |cfg|
+    comment cfg
+  end
+
+  cmd 'show running-config' do |cfg|
+    cfg.each_line
+       .reject { |line| line.match(/System Up Time/) }
+       .reject { |line| line.match(/Current System Time:/) }
+       .reject { |line| line.match(/Current SNTP Synchronized Time:/) }
+       .join
+  end
+
+  cfg :telnet, :ssh do
+    post_login do
+      if vars(:enable) == true
+        cmd 'enable'
+      elsif vars(:enable)
+        cmd 'enable', /^[pP]assword:/
+        cmd vars(:enable)
+      end
+    end
+    post_login 'terminal length 0'
+    pre_logout 'logout'
+  end
+end

data/lib/oxidized/model/eltex.rb

@@ -16,7 +16,7 @@ class Eltex < Oxidized::Model
     cfg.gsub! /^(enable (password|secret)( level \d+)? \d) .+/, '\\1 <secret hidden>'
     cfg.gsub! /^(\s+(?:password|secret)) (?:\d )?\S+/, '\\1 <secret hidden>'
     cfg.gsub! /^(tacacs-server (.+ )?key) .+/, '\\1 <secret hidden>'
-    cfg.gsub! /^((tacacs|radius) server [^\n]+\n(\s+[^\n]+\n)*\s+key) [^\n]+$/m, '\1 <secret hidden>'
+    cfg.gsub! /^((tacacs|radius) server [^\n]+\n( +[^\n]+\n)*\s+key) [^\n]+$/m, '\1 <secret hidden>'
     cfg.gsub! /username (\S+) privilege (\d+) (\S+).*/, '<secret hidden>'
     cfg.gsub! /^username \S+ password \d \S+/, '<secret hidden>'
     cfg.gsub! /^enable password \d \S+/, '<secret hidden>'

data/lib/oxidized/model/fabricos.rb

@@ -8,7 +8,7 @@ class FabricOS < Oxidized::Model
   comment '# '
 
   cmd 'chassisShow' do |cfg|
-    comment cfg.each_line.reject { |line| line.match(/Time Awake:/) || line.match(/Power Usage \(Watts\):/) || line.match(/Power Usage:/) || line.match(/Time Alive:/) || line.match(/Update:/) }.join
+    comment cfg.each_line.reject { |line| line.match(/Time Awake:/) || line.match(/Power Usage \(Watts\):/) || line.match(/Power Usage:/) || line.match(/Time Alive:/) || line.match(/Update:/) || line.match(/PS Voltage input:/) }.join
   end
 
   cmd 'configShow -all' do |cfg|

data/lib/oxidized/model/fastiron.rb

@@ -19,7 +19,9 @@ class FastIron < Oxidized::Model
     lines.each_with_index do |line, _i|
       comments << "Version: #{Regexp.last_match(1)}" if line =~ /^\s+SW: Version (.*)$/
 
-      comments << "Boot-Monitor Version: #{Regexp.last_match(1)}" if line =~ /^\s+Compressed Boot-Monitor Image size = \d+, Version:(.*)$/
+      if line =~ /^\s+Compressed Boot-Monitor Image size = \d+, Version:(.*)$/
+        comments << "Boot-Monitor Version: #{Regexp.last_match(1)}"
+      end
 
       comments << "Serial: #{Regexp.last_match(1)}" if line =~ /^\s+Serial  #:(.*)$/
     end

data/lib/oxidized/model/firelinuxos.rb

@@ -3,12 +3,21 @@ class FireLinuxOS < Oxidized::Model
 
   # Fire Linux OS is what the new FTD (FirePOWER) series devices from Cisco run. At the backend, it's mostly identical to ASA's.
 
-  prompt /^[#>]\(?.+\)?\s?/
-  comment  '! '
+  prompt /^[#>]\(?.+\)? ?$/
+  comment '! '
+
+  expect /^Syntax error: .*\n.*$/ do |data, re|
+    # The firepower does not remove the entered command, so
+    # Send CTRL-U and \n for a fresh prompt
+    send "\x15\n"
+    data.sub re, ''
+  end
 
   cmd :all do |cfg|
     cfg.gsub! /^% Invalid input detected at '\^' marker\.$|^\s+\^$/, ''
-    cfg.each_line.to_a[1..-2].join
+    # Ged rid of ANSI escape codes
+    cfg.gsub! /\e\[[0-?]*[ -\/]*[@-~]\r?/, ''
+    cfg.cut_both
   end
 
   cmd :secret do |cfg|

data/lib/oxidized/model/fortios.rb

@@ -96,7 +96,8 @@ class FortiOS < Oxidized::Model
 
     commandlist.each do |fullcmd|
       fullcfg = cmd(fullcmd)
-      next if fullcfg.lines[1..3].join =~ /(Parsing error at|command parse error)/ # Don't show for unsupported devices (e.g. FortiAnalyzer, FortiManager, FortiMail)
+      # Don't show for unsupported devices (e.g. FortiAnalyzer, FortiManager, FortiMail)
+      next if fullcfg.lines[1..3].join =~ /(Parsing error at|command parse error)/
 
       fullcfg.gsub! /(set comments "Error \(No order (found )?for (account )?ID \d+\) on).*/, '\\1 <stripped>"'
 

data/lib/oxidized/model/gaiaos.rb

@@ -26,7 +26,7 @@ class GaiaOS < Oxidized::Model
   # check for vsx / multiple context
   cmd 'show vsx' do |cfg|
     @is_vsx = cfg.include? 'VSX Enabled'
-    Oxidized.logger.debug cfg
+    logger.debug cfg
   end
 
   cmd 'show asset all' do |cfg|
@@ -46,7 +46,7 @@ class GaiaOS < Oxidized::Model
   end
 
   def single_context
-    Oxidized.logger.debug 'Single context tasks'
+    logger.debug 'Single context tasks'
     cmd 'show configuration' do |cfg|
       cfg.gsub! /^# Exported by \S+ on .*/, '# '
       cfg
@@ -54,7 +54,7 @@ class GaiaOS < Oxidized::Model
   end
 
   def multiple_context
-    Oxidized.logger.debug 'Multi context tasks'
+    logger.debug 'Multi context tasks'
     cmd 'show virtual-system all' do |systems|
       vs_items = systems.scan(/^(?<VSID>\d+)\s+(?<VSNAME>.*[^\s])/)
       allcfg = ''
@@ -62,7 +62,7 @@ class GaiaOS < Oxidized::Model
         allcfg += "\n\n\n#--------======== [ VS #{item[0]} - #{item[1]} ] ========--------\n\n"
         allcfg += "set virtual-system #{item[0]}\n\n"
         cmd "set virtual-system #{item[0]}" do |vs|
-          Oxidized.logger.debug vs
+          logger.debug vs
           cmd 'show configuration' do |vscfg|
             vscfg.gsub! /^# Exported by \S+ on .*/, '# '
             allcfg += vscfg

data/lib/oxidized/model/ios.rb

@@ -69,17 +69,25 @@ class IOS < Oxidized::Model
 
       comments << "Image:#{slave} Compiled: #{Regexp.last_match(1)}" if line =~ /^Compiled (.*)$/
 
-      comments << "Image:#{slave} Software: #{Regexp.last_match(1)}, #{Regexp.last_match(2)}" if line =~ /^(?:Cisco )?IOS .* Software,? \(([A-Za-z0-9_-]*)\), .*Version\s+(.*)$/
+      if line =~ /^(?:Cisco )?IOS .* Software,? \(([A-Za-z0-9_-]*)\), .*Version\s+(.*)$/
+        comments << "Image:#{slave} Software: #{Regexp.last_match(1)}, #{Regexp.last_match(2)}"
+      end
 
-      comments << "ROM Bootstrap: #{Regexp.last_match(3)}" if line =~ /^ROM: (IOS \S+ )?(System )?Bootstrap.*(Version.*)$/
+      if line =~ /^ROM: (IOS \S+ )?(System )?Bootstrap.*(Version.*)$/
+        comments << "ROM Bootstrap: #{Regexp.last_match(3)}"
+      end
 
       comments << "BOOTFLASH: #{Regexp.last_match(1)}" if line =~ /^BOOTFLASH: .*(Version.*)$/
 
       comments << "Memory: nvram #{Regexp.last_match(1)}" if line =~ /^(\d+[kK]) bytes of (non-volatile|NVRAM)/
 
-      comments << "Memory: flash #{Regexp.last_match(1)}" if line =~ /^(\d+[kK]) bytes of (flash memory|flash internal|processor board System flash|ATA CompactFlash)/i
+      if line =~ /^(\d+[kK]) bytes of (flash memory|flash internal|processor board System flash|ATA CompactFlash)/i
+        comments << "Memory: flash #{Regexp.last_match(1)}"
+      end
 
-      comments << "Memory: pcmcia #{Regexp.last_match(2)} #{Regexp.last_match(3)}#{Regexp.last_match(4)} #{Regexp.last_match(1)}" if line =~ /^(\d+[kK]) bytes of (Flash|ATA)?.*PCMCIA .*(slot|disk) ?(\d)/i
+      if line =~ /^(\d+[kK]) bytes of (Flash|ATA)?.*PCMCIA .*(slot|disk) ?(\d)/i
+        comments << "Memory: pcmcia #{Regexp.last_match(2)} #{Regexp.last_match(3)}#{Regexp.last_match(4)} #{Regexp.last_match(1)}"
+      end
 
       if line =~ /(\S+(?:\sseries)?)\s+(?:\(([\S ]+)\)\s+processor|\(revision[^)]+\)).*\s+with (\S+k) bytes/i
         sproc = Regexp.last_match(1)
@@ -120,7 +128,9 @@ class IOS < Oxidized::Model
     cmd cmd_line do |cfg|
       cfg = cfg.each_line.to_a[3..-1]
       cfg = cfg.reject { |line| line.match /^ntp clock-period / }.join
-      cfg = cfg.each_line.reject { |line| line.match /^! (Last|No) configuration change (at|since).*/ unless line =~ /\d+\sby\s\S+$/ }.join
+      cfg = cfg.each_line.reject do |line|
+        line.match /^! (Last|No) configuration change (at|since).*/ unless line =~ /\d+\sby\s\S+$/
+      end.join
       cfg.gsub! /^Current configuration : [^\n]*\n/, ''
       cfg.gsub! /^ tunnel mpls traffic-eng bandwidth[^\n]*\n*(
                     (?: [^\n]*\n*)*