Merge pull request getsentry#638 from eugeneius/rails_logger

Use Rails logger by default again

Author: nateberkopec

lib/raven/integrations/rails.rb

@@ -32,10 +32,13 @@ class Rails < ::Rails::Railtie
       end
     end
 
+    config.before_initialize do
+      Raven.configuration.logger = ::Rails.logger
+    end
+
     config.after_initialize do
       Raven.configure do |config|
         config.project_root ||= ::Rails.root
-        config.logger ||= ::Rails.logger
         config.release ||= config.detect_release # if project_root has changed, need to re-check
       end
 

spec/raven/integrations/rails_spec.rb

@@ -6,16 +6,22 @@
 
 describe TestApp, :type => :request do
   before(:all) do
-    Raven.configure do |config|
+    @original_configuration = Raven.configuration
+
+    Raven.configuration = Raven::Configuration.new.tap do |config|
       config.dsn = 'dummy://12345:[email protected]:3000/sentry/42'
       config.encoding = 'json'
-      config.logger = false
     end
+
     Rails.logger = Logger.new(nil)
     Rails.env = "production"
     TestApp.initialize!
   end
 
+  after(:all) do
+    Raven.configuration = @original_configuration
+  end
+
   after(:each) do
     Raven.client.transport.events = []
   end