Неопределенный метод в моем приложении. В чем дело?

Question

Я пытаюсь заблокировать IP-адреса в моем приложении следующим образом - lifeonrails.org . У меня уже есть модуль в / lib и модель banned_ip.

Почему у меня есть эта ошибка ниже от views / banned_ips / index.html?

=== Моя ошибка: ===

NoMethodError в Admin / banned_ips # index

Отображение app / views / admin / banned_ips / index.html.erb, где поднята строка # 9:

undefined method `banned?' for "127.0.0.1":String

Извлеченный источник (около строки № 9):

6:     <th>Last_ip</th>
7:     <th>Date</th>
8:   </tr>
9: <% if request.remote_ip.banned? == true %>banned<% else %>ok<% end %>
10: <% for banned_ip in @banned_ips %>
11:   <tr>
12:     <td><%=h banned_ip.first_ip %></td>

---

=== Модуль infrid.rb в / lib ===

module Infrid
  class IPAddress
    include Comparable
    def initialize(address)
      @address = address
    end
    def split
      @address.split(‘.‘).map {|s| s.to_i }
    end
    def <=>(other)
      split <=> other.split
    end
    def to_s
      @address
    end
  end
end

---

=== Модель banned_ip: ===

class BannedIp < ActiveRecord::Base
    @banned_ips # hash of ips and masks
    validates_presence_of :first_ip, :message =>"first address is needed"
    validates_presence_of :last_ip, :message =>"last address is needed"
    validates_format_of :first_ip, :with => REG_IP, :message => "is invalid (must be x.x.x.x where x is 0-255)", :if => Proc.new {|ar| !ar.first_ip.blank? }
    validates_format_of :last_ip, :with => REG_IP, :message => "is invalid (must be x.x.x.x where x is 0-255)", :if => Proc.new {|ar| !ar.last_ip.blank? }

    def self.banned?(ip)
      reload_banned_ips if @banned_ips.nil?
      begin
          ip = Infrid::IPAddress.new(ip)
          @banned_ips.each { |b|
            return true if ip.between?(b[0], b[1])
          }
      rescue
          logger.info "IP FORMAT ERROR"
          return true
      end
      false
    end
    def self.banned_ips
        reload_banned_ips if @banned_ips.nil?
        @banned_ips.collect {|b| b[0].to_s + ".." + b[1].to_s }.join"\n"
    end
    #keeps a cache of all banned ip ranges
    def self.reload_banned_ips
      r = connection.select_all("select first_ip, last_ip from banned_ips")
      if !r
        @banned_ips=[] 
      end
      @banned_ips = r.map {|item| [Infrid::IPAddress.new(item["first_ip"]),Infrid::IPAddress.new(item["last_ip"])] }
    end
end

Answer 1

Ваша проблема в том, что вы пытаетесь позвонить banned? на String, а не на BannedIp класс. У вас есть два решения.

1. Замените код для проверки на заблокированный IP на BannedIp.banned?(request.remote_ip)
2. Патч в методе к строковому классу, который вызывает метод класса для вас, который в стиле rails, но менее читаемый.

Это нужно здесь. (ошибка не позволяет блоку кода работать)

class String
  def banned?
    BannedIp.banned?(self)
  end
end

Answer 2

То, что request.remote_ip возвращает IP-адрес в виде строки, а строки не имеют метода banned?. Похоже, вы хотите BannedIP.banned?(request.remote_ip).