Respect the :auto_reconnect setting for AMI

adhearsion · Oct 2, 2010 · 1748f7f · eric · Oct 3, 2010 · bklang
1 parent 111106c
commit 1748f7f
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 7 deletions.
diff --git a/lib/adhearsion/initializer/asterisk.rb b/lib/adhearsion/initializer/asterisk.rb
@@ -50,7 +50,7 @@ def manager_interface
         end
 
         def ami_options
-          %w(host port username password events).inject({}) do |options, property|
+          %w(host port username password events auto_reconnect).inject({}) do |options, property|
             options[property.to_sym] = config.ami.send property
             options
           end

diff --git a/lib/adhearsion/initializer/configuration.rb b/lib/adhearsion/initializer/configuration.rb
@@ -129,8 +129,8 @@ def default_listening_host
       end
 
       def initialize(overrides = {})
-        @listening_port = overrides.has_key?(:port) ? overrides.delete(:port) : self.class.default_listening_port
         @listening_host = overrides.has_key?(:host) ? overrides.delete(:host) : self.class.default_listening_host
+        @listening_port = overrides.has_key?(:port) ? overrides.delete(:port) : self.class.default_listening_port
         super
       end
     end
@@ -158,7 +158,7 @@ def initialize(overrides = {})
       end
 
       class AMIConfiguration < AbstractConfiguration
-        attr_accessor :port, :username, :password, :events, :host
+        attr_accessor :port, :username, :password, :events, :host, :auto_reconnect
 
         class << self
           def default_port
@@ -172,12 +172,17 @@ def default_events
           def default_host
             'localhost'
           end
+
+          def default_auto_reconnect
+            true
+          end
         end
 
         def initialize(overrides = {})
-          self.host   = self.class.default_host
-          self.port   = self.class.default_port
-          self.events = self.class.default_events
+          self.host           = self.class.default_host
+          self.port           = self.class.default_port
+          self.events         = self.class.default_events
+          self.auto_reconnect = self.class.default_auto_reconnect
           super
         end
       end

diff --git a/lib/adhearsion/voip/asterisk/manager_interface.rb b/lib/adhearsion/voip/asterisk/manager_interface.rb
@@ -108,8 +108,9 @@ def causal_event_terminator_name_for(action_name)
           # @param [Hash] options Available options are :host, :port, :username, :password, and :events
           #
           def initialize(options={})
+puts options.inspect
             options = parse_options options
-
+puts options.inspect
             @host           = options[:host]
             @username       = options[:username]
             @password       = options[:password]