nexpose 0.9.8 → 1.0.0

data/lib/nexpose/discovery.rb

@@ -27,20 +27,22 @@ module Nexpose
     end
   end
 
-  class DiscoveryConnection
+  class DiscoveryConnection < APIObject
     include XMLUtils
 
     module Protocol
-      HTTP = 'HTTP'
+      HTTP  = 'HTTP'
       HTTPS = 'HTTPS'
-      LDAP = 'LDAP'
+      LDAP  = 'LDAP'
       LDAPS = 'LDAPS'
     end
 
     module Type
-      VSPHERE = 'VSPHERE'
-      AWS = 'AWS'
-      ACTIVESYNC = 'ACTIVESYNC'
+      VSPHERE               = 'VSPHERE'
+      AWS                   = 'AWS'
+      ACTIVESYNC            = 'ACTIVESYNC'
+      ACTIVESYNC_POWERSHELL = 'ACTIVESYNC_POWERSHELL'
+      ACTIVESYNC_OFFICE365  = 'ACTIVESYNC_OFFICE365'
     end
 
     # A unique identifier for this connection.
@@ -55,18 +57,30 @@ module Nexpose
     # The IP address or fully qualified domain name of the server.
     attr_accessor :address
 
+    # The engine ID to use for this connection.
+    attr_accessor :engine_id
+
     # A user name that can be used to log into the server.
     attr_accessor :user
 
     # The password to use when connecting with the defined user.
     attr_accessor :password
 
-    # The protocol used for conneting to the server. One of DiscoveryConnection::Protocol
+    # The protocol used for connecting to the server. One of DiscoveryConnection::Protocol
     attr_accessor :protocol
 
     # The port used for connecting to the server. A valid port from 1 to 65535.
     attr_accessor :port
 
+    # The hostname of the exchange server to connect for exchange powershell connections
+    attr_accessor :exchange_hostname
+
+    # The exchange username to connect for exchange powershell connections
+    attr_accessor :exchange_username
+
+    # The exchange password to connect for exchange powershell connections
+    attr_accessor :exchange_password
+
     # Whether or not the connection is active.
     # Discovery is only possible when the connection is active.
     attr_accessor :status
@@ -79,7 +93,7 @@ module Nexpose
     # @param [String] user User name for credentials on this connection.
     # @param [String] password Password for credentials on this connection.
     #
-    def initialize(name, address, user, password = nil)
+    def initialize(name = nil, address = nil, user = nil, password = nil)
       @name, @address, @user, @password = name, address, user, password
       @type = nil  # for backwards compatibilitly, at some point should set this to Type::VSPHERE
       @id = -1
@@ -143,13 +157,17 @@ module Nexpose
 
     def as_xml
       xml = REXML::Element.new('DiscoveryConnection')
-      xml.attributes['name']      = @name
-      xml.attributes['address']   = @address
-      xml.attributes['port']      = @port
-      xml.attributes['protocol']  = @protocol
-      xml.attributes['user-name'] = @user
-      xml.attributes['password']  = @password
-      xml.attributes['type']      = @type if @type
+      xml.attributes['name']              = @name
+      xml.attributes['address']           = @address
+      xml.attributes['port']              = @port
+      xml.attributes['protocol']          = @protocol
+      xml.attributes['user-name']         = @user
+      xml.attributes['password']          = @password
+      xml.attributes['exchange-hostname'] = @exchange_hostname if @exchange_hostname
+      xml.attributes['exchange-username'] = @exchange_username if @exchange_username
+      xml.attributes['exchange-password'] = @exchange_password if @exchange_password
+      xml.attributes['type']              = @type if @type
+      xml.attributes['engine-id'] = @engine_id if @engine_id && @engine_id != -1
       xml
     end
 
@@ -167,6 +185,70 @@ module Nexpose
       conn.status = xml.attributes['connection-status']
       conn
     end
+
+    def to_json
+      JSON.generate(to_h)
+    end
+
+    def to_h
+      { id: id,
+        name: name,
+        type: type
+        # TODO Add remaining instance fields, once it is introduced in resource object
+      }
+    end
+
+    def ==(other)
+      eql?(other)
+    end
+
+    def eql?(other)
+      id.eql?(other.id) &&
+      name.eql?(other.name) &&
+      type.eql?(other.type)
+      # TODO Add remaining instance fields, once it is introduced in resource object
+    end
+
+    # Override of filter criterion to account for proper JSON naming.
+    #
+    class Criterion < Nexpose::Criterion
+      # Convert to Hash, which can be converted to JSON for API calls.
+      def to_h
+        { operator: operator,
+          values: Array(value),
+          field_name: field }
+      end
+
+      # Create a Criterion object from a JSON-derived Hash.
+      #
+      # @param [Hash] json JSON-derived Hash of a Criterion object.
+      # @return [Criterion] Parsed object.
+      #
+      def self.parseHash(hash)
+        Criterion.new(hash[:field_name],
+                      hash[:operator],
+                      hash[:values])
+      end
+    end
+
+    # Override of filter criteria to account for different parsing from JSON.
+    #
+    class Criteria < Nexpose::Criteria
+      # Create a Criteria object from a Hash.
+      #
+      # @param [Hash] Hash of a Criteria object.
+      # @return [Criteria] Parsed object.
+      #
+      def self.parseHash(hash)
+        # The call returns empty JSON, so default to 'AND' if not present.
+        operator = hash[:operator] || 'AND'
+        ret = Criteria.new([], operator)
+        hash[:criteria].each do |c|
+          ret.criteria << Criterion.parseHash(c)
+        end
+        ret
+      end
+    end
   end
 
   class DiscoveredAsset
@@ -216,8 +298,51 @@ module Nexpose
       @name, @protocol, @address, @user, @password = name, protocol, address, user, password
       @type = Type::ACTIVESYNC
       @id = -1
-      @port = 443   #port not used for mobile connection
+      @port = 443 # port not used for mobile connection
+    end
+  end
+  
+  class MobilePowershellDiscoveryConnection < DiscoveryConnection
+    # Create a new Mobile Powershell discovery connection.
+    #
+    # @param [String] name Name to assign to this connection.
+    # @param [String] address IP or fully qualified domain name of the
+    #    WinRM server.
+    # @param [String] user WinRM User name for credentials on this connection.
+    # @param [String] password WinRM password for credentials on this connection.
+    # @param [String] exchange_hostname fully qualified domain name of the exchange server
+    # @param [String] exchange_username Exchange User name for exchange credentials on this connection.
+    # @param [String] exchange_password Exchange password for exchange credentials on this connection.
+    #
+    def initialize(name, address, user, password, exchange_hostname, exchange_username, exchange_password)
+      @name, @address, @user, @password = name, address, user, password
+      @protocol = Protocol::HTTPS
+      @exchange_hostname, @exchange_username, @exchange_password = exchange_hostname, exchange_username, exchange_password
+      @type = Type::ACTIVESYNC_POWERSHELL
+      @id = -1
+      @port = 443 # port not used for mobile connection
     end
+  end
 
+  class MobileOffice365DiscoveryConnection < DiscoveryConnection
+    # Create a new Mobile Office365 discovery connection.
+    #
+    # @param [String] name Name to assign to this connection.
+    # @param [String] address IP or fully qualified domain name of the
+    #    WinRM server.
+    # @param [String] user WinRM User name for credentials on this connection.
+    # @param [String] password WinRM password for credentials on this connection.
+    # @param [String] exchange_username Exchange User name for exchange credentials on this connection.
+    # @param [String] exchange_password Exchange password for exchange credentials on this connection.
+    #
+    def initialize(name, address, user, password, exchange_username, exchange_password)
+      @name, @address, @user, @password = name, address, user, password
+      @protocol = Protocol::HTTPS
+      @exchange_hostname = '' # nexpose will set to office365 server
+      @exchange_username, @exchange_password = exchange_username, exchange_password
+      @type = Type::ACTIVESYNC_OFFICE365
+      @id = -1
+      @port = 443 # port not used for mobile connection
+    end
   end
 end

data/lib/nexpose/discovery/filter.rb

@@ -10,10 +10,10 @@ module Nexpose
       DATACENTER ='DATACENTER'
 
       # Valid Operators: CONTAINS, NOT_CONTAINS
-      GUEST_OS_FAMILY = 'GUEST_OS_FAMILY'
+      GUEST_OS_FAMILY = 'GUEST_OS_FAMILY'   #Also AWS Filter
 
       # Valid Operators: IN, NOT_IN
-      IP_ADDRESS_RANGE = 'IP_ADDRESS'
+      IP_ADDRESS_RANGE = 'IP_ADDRESS'   #Also AWS Filter
 
       # Valid Operators: IN, NOT_IN
       # Valid Values (See Value::PowerState): ON, OFF, SUSPENDED
@@ -25,13 +25,36 @@ module Nexpose
       # Valid Operators: IS, IS_NOT, CONTAINS, NOT_CONTAINS, STARTS_WITH
       VIRTUAL_MACHINE_NAME = 'VM'
 
-      ###### Mobile Filters ######
+      # valid Operators: IS, IS_NOT, CONTAINS, NOT_CONTAINS, STARTS_WITH
+      HOST = 'HOST_NAME'
+
+      ###### AWS Filters ######
+      # Valid Operators: CONTAINS, NOT_CONTAINS
+      AVAILABILITY_ZONE = 'AVAILABILITY_ZONE'
+
+      # Valid Operators: CONTAINS, NOT_CONTAINS
+      INSTANCE_ID = 'INSTANCE_ID'
+
+      # Valid Operators: IS, IS_NOT, CONTAINS, NOT_CONTAINS, STARTS_WITH
+      INSTANCE_NAME = 'INSTANCE_NAME'
+
+      # Valid Operators: IN, NOT_IN
+      INSTANCE_STATE = 'INSTANCE_STATE'
+
+      # Valid Operators: IN, NOT_IN
+      INSTANCE_TYPE = 'INSTANCE_TYPE'
+
+      # Valid Operators: IN, NOT_IN
+      REGION ='REGION'
+
+      ###### Mobile or Active sync Filters ######
       # Valid Operators: CONTAINS, NOT_CONTAINS
       OPERATING_SYSTEM = 'DEVICE_OPERATING_SYSTEM'
 
       # Valid Operators: IS, IS_NOT, CONTAINS, NOT_CONTAINS, STARTS_WITH
       USER = 'DEVICE_USER_DISPLAY_NAME'
 
+
     end
 
     module Value

data/lib/nexpose/engine.rb

@@ -16,6 +16,22 @@ module Nexpose
       response.success
     end
 
+    # Reverses the direction of a connection to an engine
+    # If the connection is currently initiated from the console this method
+    # will have the engine initiate the connection. If the connection is
+    # currently initiated by the engine this method with initiate the connection
+    # from the console instead. Requires a restart of the console for the
+    # connection to be properly established.
+    #
+    # @param [Fixnum] engine_id Unique ID of the engine.
+    # @return [Boolean] true if the connection is successfully reversed.
+    #
+    def reverse_engine_connection(engine_id)
+      uri = "/api/2.1/engine/#{engine_id}/reverseConnection"
+      response = AJAX.put(self, uri)
+      response.eql?("true")
+    end
+
     # Provide a list of current scan activities for a specific Scan Engine.
     #
     # @return [Array[ScanSummary]] Array of ScanSummary objects associated with

data/lib/nexpose/json_serializer.rb

@@ -0,0 +1,92 @@
+module Nexpose
+  module JsonSerializer
+    @@namespace = 'Nexpose'
+
+    def deserialize(data)
+      data.each do |key, value|
+        if respond_to?(key)
+          property = value
+
+          if value.respond_to? :each
+            obj = resolve_type(key)
+
+            unless obj.nil?
+              if value.is_a?(Array)
+                property = value.map { |dv|  ((dv.respond_to? :each) ? create_object(obj, dv).deserialize(dv): dv) }
+              else
+                property = create_object(obj, value).deserialize(value)
+              end
+            end
+          elsif value.is_a?(String) && value.match(/^\d{8}T\d{6}\.\d{3}/)
+            property = ISO8601.to_time(value)
+          end
+
+          instance_variable_set("@#{key}", property)
+        end
+      end
+
+      self
+    end
+
+    def serialize()
+      hash = to_hash(Hash.new)
+
+      unless hash.nil?
+        JSON.generate(hash)
+      end
+    end
+
+    def to_hash(hash)
+      self.instance_variables.each do |m|
+        value = self.instance_variable_get(m)
+        hash[m.to_s.delete('@')] = do_hash(value)
+      end
+
+      hash
+    end
+
+    private
+
+    def do_hash(obj)
+      if obj.is_a?(Array)
+        obj = obj.map do |el|
+          do_hash(el)
+
+        end
+      elsif obj.class.included_modules.include? JsonSerializer
+        obj = obj.to_hash(Hash.new)
+      end
+
+      obj
+    end
+
+    def create_object(obj, data)
+      if obj.respond_to?(:json_initializer)
+        obj.method(:json_initializer).call(data)
+      else
+        obj.method(:new).call
+      end
+    end
+
+    def resolve_type(field)
+      class_name = normalize_field(field)
+      type_attribute = "#{field}_type"
+
+      if self.respond_to?(type_attribute)
+        clazz = self.public_send(type_attribute)
+      elsif Object.const_get(@@namespace).const_defined?(class_name)
+        resolved = Object.const_get(@@namespace).const_get(class_name)
+        clazz = resolved if resolved.included_modules.include? JsonSerializer
+      end
+
+      clazz
+    end
+
+    def normalize_field(field)
+      class_name = field.to_s.split('_').map(&:capitalize!).join
+      class_name = 'Vulnerability' if class_name == 'Vulnerabilities'
+      class_name.chop! if class_name.end_with?('s')
+      class_name
+    end
+  end
+end

data/lib/nexpose/scan.rb

@@ -11,6 +11,16 @@ module Nexpose
       scan_devices([device])
     end
 
+    # Perform an ad hoc scan of a single device at a specific time.
+    #
+    # @param [Device] device Device to scan.
+    # @param [Array[adhoc_schedules]] list of scheduled times at which to run
+    # @return [Status] whether the request was successful
+    #
+    def scan_device_with_schedule(device, schedule)
+      scan_devices_with_schedule([device], schedule)
+    end
+
     # Perform an ad hoc scan of a subset of devices for a site.
     # Nexpose only allows devices from a single site to be submitted per
     # request.
@@ -24,17 +34,45 @@ module Nexpose
     # @return [Scan] Scan launch information.
     #
     def scan_devices(devices)
-      site_id = devices.map { |d| d.site_id }.uniq.first
-      xml = make_xml('SiteDevicesScanRequest', { 'site-id' => site_id })
+      site_id = devices.map(&:site_id).uniq.first
+      xml = make_xml('SiteDevicesScanRequest', 'site-id' => site_id)
       elem = REXML::Element.new('Devices')
       devices.each do |device|
-        elem.add_element('device', { 'id' => "#{device.id}" })
+        elem.add_element('device', 'id' => "#{device.id}")
       end
       xml.add_element(elem)
 
       _scan_ad_hoc(xml)
     end
 
+    # Perform an ad hoc scan of a subset of devices for a site.
+    # Nexpose only allows devices from a single site to be submitted per
+    # request.
+    # Method is designed to take objects from a Device listing.
+    #
+    # For example:
+    #   devices = nsc.devices(5)
+    #   nsc.scan_devices(devices.take(10))
+    #
+    # @param [Array[Device]] devices List of devices to scan.
+    # @param [Array[adhoc_schedules]] list of scheduled times at which to run
+    # @return [Status] whether the request was successful
+    #
+    def scan_devices_with_schedule(devices, schedules)
+      site_id = devices.map(&:site_id).uniq.first
+      xml = make_xml('SiteDevicesScanRequest', 'site-id' => site_id)
+      elem = REXML::Element.new('Devices')
+      devices.each do |device|
+        elem.add_element('device', 'id' => "#{device.id}")
+      end
+      xml.add_element(elem)
+      scheds = REXML::Element.new('Schedules')
+      schedules.each { |sched| scheds.add_element(sched.as_xml) }
+      xml.add_element(scheds)
+
+      _scan_ad_hoc_with_schedules(xml)
+    end
+
     # Perform an ad hoc scan of a single asset of a site.
     #
     # @param [Fixnum] site_id Site ID that the assets belong to.
@@ -45,6 +83,17 @@ module Nexpose
       scan_assets(site_id, [asset])
     end
 
+    # Perform an ad hoc scan of a single asset of a site at a specific time
+    #
+    # @param [Fixnum] site_id Site ID that the assets belong to.
+    # @param [HostName|IPRange] asset Asset to scan.
+    # @param [Array[adhoc_schedules]] list of scheduled times at which to run
+    # @return [Status] whether the request was successful
+    #
+    def scan_asset_with_schedule(site_id, asset, schedule)
+      scan_assets_with_schedule(site_id, [asset], schedule)
+    end
+
     # Perform an ad hoc scan of a subset of assets for a site.
     # Only assets from a single site should be submitted per request.
     # Method is designed to take objects filtered from Site#assets.
@@ -58,7 +107,7 @@ module Nexpose
     # @return [Scan] Scan launch information.
     #
     def scan_assets(site_id, assets)
-      xml = make_xml('SiteDevicesScanRequest', { 'site-id' => site_id })
+      xml = make_xml('SiteDevicesScanRequest', 'site-id' => site_id)
       hosts = REXML::Element.new('Hosts')
       assets.each { |asset| _append_asset!(hosts, asset) }
       xml.add_element(hosts)
@@ -66,6 +115,59 @@ module Nexpose
       _scan_ad_hoc(xml)
     end
 
+    # Perform an ad hoc scan of a subset of assets for a site by adding a specific runtime.
+    # Only assets from a single site should be submitted per request.
+    # Method is designed to take objects filtered from Site#assets.
+    #
+    # For example:
+    #   site = Site.load(nsc, 5)
+    #   nsc.scan_assets_with_schedule(5, site.assets.take(10), schedules)
+    #
+    # @param [Fixnum] site_id Site ID that the assets belong to.
+    # @param [Array[HostName|IPRange]] assets List of assets to scan.
+    # @param [Array[adhoc_schedules]] list of scheduled times at which to run
+    # @return [Status] whether the request was successful
+    #
+    def scan_assets_with_schedule(site_id, assets, schedules)
+      xml = make_xml('SiteDevicesScanRequest', 'site-id' => site_id)
+      hosts = REXML::Element.new('Hosts')
+      assets.each { |asset| _append_asset!(hosts, asset) }
+      xml.add_element(hosts)
+      scheds = REXML::Element.new('Schedules')
+      schedules.each { |sched| scheds.add_element(sched.as_xml) }
+      xml.add_element(scheds)
+
+      _scan_ad_hoc_with_schedules(xml)
+    end
+
+    # Perform an ad hoc scan of a subset of IP addresses for a site at a specific time.
+    # Only IPs from a single site can be submitted per request,
+    # and IP addresses must already be included in the site configuration.
+    # Method is designed for scanning when the targets are coming from an
+    # external source that does not have access to internal identfiers.
+    #
+    # For example:
+    #   to_scan = ['192.168.2.1', '192.168.2.107']
+    #   nsc.scan_ips(5, to_scan)
+    #
+    # @param [Fixnum] site_id Site ID that the assets belong to.
+    # @param [Array[String]] ip_addresses Array of IP addresses to scan.
+    # @return [Status] whether the request was successful
+    #
+    def scan_ips_with_schedule(site_id, ip_addresses, schedules)
+      xml = make_xml('SiteDevicesScanRequest', 'site-id' => site_id)
+      hosts = REXML::Element.new('Hosts')
+      ip_addresses.each do |ip|
+        xml.add_element('range', 'from' => ip)
+      end
+      xml.add_element(hosts)
+      scheds = REXML::Element.new('Schedules')
+      schedules.each { |sched| scheds.add_element(sched.as_xml) }
+      xml.add_element(scheds)
+
+      _scan_ad_hoc_with_schedules(xml)
+    end
+
     # Perform an ad hoc scan of a subset of IP addresses for a site.
     # Only IPs from a single site can be submitted per request,
     # and IP addresses must already be included in the site configuration.
@@ -81,10 +183,10 @@ module Nexpose
     # @return [Scan] Scan launch information.
     #
     def scan_ips(site_id, ip_addresses)
-      xml = make_xml('SiteDevicesScanRequest', { 'site-id' => site_id })
+      xml = make_xml('SiteDevicesScanRequest', 'site-id' => site_id)
       hosts = REXML::Element.new('Hosts')
       ip_addresses.each do |ip|
-        xml.add_element('range', { 'from' => ip })
+        xml.add_element('range', 'from' => ip)
       end
       xml.add_element(hosts)
 
@@ -97,7 +199,7 @@ module Nexpose
     # @return [Scan] Scan launch information.
     #
     def scan_site(site_id)
-      xml = make_xml('SiteScanRequest', { 'site-id' => site_id })
+      xml = make_xml('SiteScanRequest', 'site-id' => site_id)
       response = execute(xml)
       Scan.parse(response.res) if response.success
     end
@@ -110,7 +212,7 @@ module Nexpose
     #
     def _append_asset!(xml, asset)
       if asset.is_a? Nexpose::IPRange
-        xml.add_element('range', { 'from' => asset.from, 'to' => asset.to })
+        xml.add_element('range', 'from' => asset.from, 'to' => asset.to)
       else  # Assume HostName
         host = REXML::Element.new('host')
         host.text = asset.host
@@ -129,6 +231,15 @@ module Nexpose
       Scan.parse(r.res)
     end
 
+    # Utility method for executing prepared XML for adhoc with schedules
+    #
+    # @param [REXML::Document] xml Prepared API call to execute.
+    #
+    def _scan_ad_hoc_with_schedules(xml)
+      r = execute(xml, '1.1', timeout: 60)
+      r.success
+    end
+
     # Stop a running or paused scan.
     #
     # @param [Fixnum] scan_id ID of the scan to stop.
@@ -136,7 +247,7 @@ module Nexpose
     #   updated.
     #
     def stop_scan(scan_id, wait_sec = 0)
-      r = execute(make_xml('ScanStopRequest', { 'scan-id' => scan_id }))
+      r = execute(make_xml('ScanStopRequest', 'scan-id' => scan_id))
       if r.success
         so_far = 0
         while so_far < wait_sec
@@ -155,7 +266,7 @@ module Nexpose
     # @return [String] Current status of the scan. See Nexpose::Scan::Status.
     #
     def scan_status(scan_id)
-      r = execute(make_xml('ScanStatusRequest', { 'scan-id' => scan_id }))
+      r = execute(make_xml('ScanStatusRequest', 'scan-id' => scan_id))
       r.success ? r.attributes['status'] : nil
     end
 
@@ -164,7 +275,7 @@ module Nexpose
     # @param [Fixnum] scan_id The scan ID.
     #
     def resume_scan(scan_id)
-      r = execute(make_xml('ScanResumeRequest', { 'scan-id' => scan_id }), '1.1', timeout: 60)
+      r = execute(make_xml('ScanResumeRequest', 'scan-id' => scan_id), '1.1', timeout: 60)
       r.success ? r.attributes['success'] == '1' : false
     end
 
@@ -173,7 +284,7 @@ module Nexpose
     # @param [Fixnum] scan_id The scan ID.
     #
     def pause_scan(scan_id)
-      r = execute(make_xml('ScanPauseRequest', { 'scan-id' => scan_id }))
+      r = execute(make_xml('ScanPauseRequest', 'scan-id' => scan_id))
       r.success ? r.attributes['success'] == '1' : false
     end
 
@@ -218,7 +329,7 @@ module Nexpose
     # @return [ScanSummary] ScanSummary object providing statistics for the scan.
     #
     def scan_statistics(scan_id)
-      r = execute(make_xml('ScanStatisticsRequest', { 'scan-id' => scan_id }))
+      r = execute(make_xml('ScanStatisticsRequest', 'scan-id' => scan_id))
       if r.success
         ScanSummary.parse(r.res.elements['//ScanSummary'])
       else
@@ -353,6 +464,7 @@ module Nexpose
     def initialize(scan_id, site_id, engine_id, status, start_time, end_time)
       @scan_id, @site_id, @engine_id, @status, @start_time, @end_time = scan_id, site_id, engine_id, status, start_time, end_time
     end
+
     def self.parse(xml)
       # Start time can be empty in some error conditions.
       start_time = nil
@@ -382,7 +494,6 @@ module Nexpose
   # Object that represents a summary of a scan.
   #
   class ScanSummary < ScanData
-
     # The reason the scan was stopped or failed, if applicable.
     attr_reader :message
 
@@ -408,7 +519,7 @@ module Nexpose
       tasks = Tasks.parse(xml.elements['tasks'])
       nodes = Nodes.parse(xml.elements['nodes'])
       vulns = Vulnerabilities.parse(xml.attributes['scan-id'], xml)
-      msg = xml.elements['message'] ?  xml.elements['message'].text : nil
+      msg = xml.elements['message'] ? xml.elements['message'].text : nil
 
       # Start time can be empty in some error conditions.
       start_time = nil
@@ -440,7 +551,6 @@ module Nexpose
     # Value class to tracking task counts.
     #
     class Tasks
-
       attr_reader :pending, :active, :completed
 
       def initialize(pending, active, completed)
@@ -463,7 +573,6 @@ module Nexpose
     # Value class for tracking node counts.
     #
     class Nodes
-
       attr_reader :live, :dead, :filtered, :unresolved, :other
 
       def initialize(live, dead, filtered, unresolved, other)
@@ -488,7 +597,6 @@ module Nexpose
     # Value class for tracking vulnerability counts.
     #
     class Vulnerabilities
-
       attr_reader :vuln_exploit, :vuln_version, :vuln_potential,
                   :not_vuln_exploit, :not_vuln_version,
                   :error, :disabled, :other
@@ -497,11 +605,11 @@ module Nexpose
                      not_vuln_exploit, not_vuln_version,
                      error, disabled, other)
         @vuln_exploit, @vuln_version, @vuln_potential,
-          @not_vuln_exploit, @not_vuln_version,
-          @error, @disabled, @other =
-          vuln_exploit, vuln_version, vuln_potential,
-          not_vuln_exploit, not_vuln_version,
-          error, disabled, other
+            @not_vuln_exploit, @not_vuln_version,
+            @error, @disabled, @other =
+            vuln_exploit, vuln_version, vuln_potential,
+            not_vuln_exploit, not_vuln_version,
+            error, disabled, other
       end
 
       # Parse REXML to Vulnerabilities object.