Method log_halt has a Cognitive Complexity of 12 (exceeds 5 allowed). Consider refactoring.

Open

    def log_halt code=nil, exception=nil
      message = exception.is_a?(String) ? exception : exception.to_s
      begin
        if block_given?
          return yield

• Read up
  Read up

Method loadSubnetData has 34 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    def loadSubnetData subnet
      super
      conf = format((@config+@leases).split("\n"))
      # scan for host statements
      conf.scan(/host\s+(\S+\s*\{[^}]+\})/) do |host|

Method parse_record_options has 33 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    def parse_record_options text
      options = {}
      case text
        # standard record values
      when /^hardware\s+ethernet\s+(\S+)/

Method run! has 33 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    def self.run!(options={})
      set options
      handler      = detect_rack_handler
      handler_name = handler.name.gsub(/.*::/, '')

Method unused_ip has a Cognitive Complexity of 11 (exceeds 5 allowed). Consider refactoring.

Open

    def unused_ip args = {}
      # first check if we already have a record for this host
      # if we do, we can simply reuse the same ip address.
      if args[:mac] and r=has_mac?(args[:mac])
        logger.debug "Found an existing dhcp record #{r}, reusing..."

• Read up
  Read up

Method find_puppetca has a Cognitive Complexity of 11 (exceeds 5 allowed). Consider refactoring.

Open

    def find_puppetca
      ssl_dir = Pathname.new ssldir
      unless (ssl_dir + "ca").directory?
        logger.warn "PuppetCA: SSL/CA unavailable on this machine: ssldir not found at #{ssl_dir}"
        raise "SSL/CA unavailable on this machine"

• Read up
  Read up

Method addRecord has a Cognitive Complexity of 11 (exceeds 5 allowed). Consider refactoring.

Open

    def addRecord options={}
      record = super(options)

      cmd = "scope #{record.subnet.network} add reservedip #{record.ip} #{record.mac.gsub(/:/,"")} #{record.name}"
      execute(cmd, "Added DHCP reservation for #{record}")

• Read up
  Read up

Method unused_ip has 30 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    def unused_ip args = {}
      # first check if we already have a record for this host
      # if we do, we can simply reuse the same ip address.
      if args[:mac] and r=has_mac?(args[:mac])
        logger.debug "Found an existing dhcp record #{r}, reusing..."

Method tcp_pingable? has 30 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    def tcp_pingable? ip
      # This code is from net-ping, and stripped down for use here
      # We don't need all the ldap dependencies net-ping brings in

      @service_check = true

Method daemon has a Cognitive Complexity of 10 (exceeds 5 allowed). Consider refactoring.

Open

    def self.daemon(nochdir = nil, noclose = nil)
      exit if fork                     # Parent exits, child continues.
      Process.setsid                   # Become session leader.
      exit if fork                     # Zap session leader. See [1].

• Read up
  Read up

Method bmc_setup has 27 lines of code (exceeds 25 allowed). Consider refactoring.

Open

  def bmc_setup

    raise "Smart Proxy is not configured to support BMC control" unless SETTINGS.bmc

    # Either use the default provider or allow user to specify provider in request

Method execute has a Cognitive Complexity of 9 (exceeds 5 allowed). Consider refactoring.

Open

    def execute cmd, msg=nil, error_only=false
      tsecs = 5
      response = nil
      interpreter = SETTINGS.x86_64 ? 'c:\windows\sysnative\cmd.exe' : 'c:\windows\system32\cmd.exe'
      command  = interpreter + ' /c c:\Windows\System32\netsh.exe -c dhcp ' + "server #{name} #{cmd}"

• Read up
  Read up

Method read_config has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

    def read_config file
      logger.debug "Reading config file #{file}"
      config = []
      File.readlines(file).each do |line|
        if /^include\s+"(.*)"\s*;/ =~ line.strip

• Read up
  Read up

Method nsupdate has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

    def nsupdate cmd
      status = nil
      if cmd == "connect"
        find_nsupdate if @nsupdate.nil?
        nsupdate_cmd = "#{@nsupdate} #{nsupdate_args}"

• Read up
  Read up

Method find_subnet has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

    def find_subnet value
      subnets.each do |s|
        return s if value.is_a?(String) and s.network == value
        return s if value.is_a?(Proxy::DHCP::Record) and s.include?(value.ip)
        return s if value.is_a?(IPAddr) and s.include?(value)

• Read up
  Read up

Method valid_range has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

    def valid_range args = {}
      logger.debug "trying to find an ip address, we got #{args.inspect}"
      if args[:from] and (from=validate_ip(args[:from])) and args[:to] and (to=validate_ip(args[:to]))
        raise Proxy::DHCP::Error, "Range does not belong to provided subnet" unless self.include?(from) and self.include?(to)
        from = IPAddr.new(from)

• Read up
  Read up

Method load has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

      def load
        Puppet.clear
        if Puppet::PUPPETVERSION.to_i >= 3
          # Used on Puppet 3.0, private method that clears the "initialized or
          # not" state too, so a full config reload takes place and we pick up

• Read up
  Read up

Method initialize has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.

Open

    def initialize(cmd)
      # clean finished tasks from the array
      @@tasks = @@tasks.collect { |t| nil unless t.is_a? String }.compact

      # run the task in its own thread

• Read up
  Read up

Method loadSubnets has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.

Open

    def loadSubnets
      super
      cmd = "show scope"
      msg = "Enumerated the scopes on #{@name}"

• Read up
  Read up

Method addRecord has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.

Open

    def addRecord options = {}
      ip = validate_ip options[:ip]
      mac = validate_mac options[:mac]
      name = options[:hostname] || raise(Proxy::DHCP::Error, "Must provide hostname")
      net = options.delete("network")

• Read up
  Read up