- move external requires into 'register' method

lib/logstash/filters/geoip.rb

@@ -1,6 +1,5 @@
 require "logstash/filters/base"
 require "logstash/namespace"
-require "geoip"
 
 # Add GeoIP fields from Maxmind database
 #
@@ -28,6 +27,7 @@ class LogStash::Filters::GeoIP < LogStash::Filters::Base
 
   public
   def register
+    require "geoip"
     if @database.nil?
       if __FILE__ =~ /^file:\/.+!.+/
         # Running from a jar, assume GeoLiteCity.dat is at the root.

lib/logstash/filters/xml.rb

@@ -1,6 +1,5 @@
 require "logstash/filters/base"
 require "logstash/namespace"
-require "nokogiri"
 
 # XML filter. Takes a field that contains XML and expands it into
 # an actual datastructure.
@@ -42,6 +41,7 @@ class LogStash::Filters::Xml < LogStash::Filters::Base
 
   public
   def register
+    require "nokogiri"
     require "xmlsimple"
     @xml = {}
 

lib/logstash/inputs/file.rb

@@ -4,8 +4,6 @@ require "logstash/namespace"
 require "pathname"
 require "socket" # for Socket.gethostname
 
-require "addressable/uri"
-
 # Stream events from files.
 #
 # By default, each event is assumed to be one line. If you
@@ -72,6 +70,7 @@ class LogStash::Inputs::File < LogStash::Inputs::Base
 
   public
   def register
+    require "addressable/uri"
     require "filewatch/tail"
     require "digest/md5"
     LogStash::Util::set_thread_name("input|file|#{path.join(":")}")