rollbar 0.11.2 → 0.11.3

checksums.yaml

@@ -0,0 +1,7 @@
+---
+SHA1:
+  metadata.gz: a05a8983a4b7621ec27fb68f6a8181357f709f5b
+  data.tar.gz: cad9b0ab2cf6ca8a05dcda83aeff733e9c8ee6ad
+SHA512:
+  metadata.gz: e63b34f583fd421501c41d60df6f597ee0acd7656704e251a42af46923450140e87759ac1414ff3d64cebe3e906ce2a45dfcd20e4f2ed0f6a6dd974dba01c8e4
+  data.tar.gz: a034f48d725e87bc5b7013698a5e30e78378fca32aa276558582c72885faafefddcce6442c8a663e6622f9830bbecf3080d220da2eaa1605057dedfb2d4d9f43

data/CHANGELOG.md

@@ -1,5 +1,8 @@
 # Change Log
 
+**0.11.3**
+- Make sure the environment is valid at item sending time so that it isn't set incorrectly during configuration
+
 **0.11.2**
 - Test rake task now works properly if force_ssl is true
 

data/lib/generators/rollbar/templates/initializer.rb

@@ -31,7 +31,7 @@ Rollbar.configure do |config|
   #
   # You can also specify a callable, which will be called with the exception instance.
   # config.exception_level_filters.merge!('MyCriticalException' => lambda { |e| 'critical' })
-  
+
   # Enable asynchronous reporting (uses girl_friday or Threading if girl_friday
   # is not installed)
   # config.use_async = true

data/lib/rollbar.rb

@@ -340,9 +340,15 @@ module Rollbar
 
     def base_data(level = 'error')
       config = configuration
+      
+      environment = config.environment
+      if environment.nil? || environment.empty?
+        environment = 'unspecified'
+      end
+      
       data = {
         :timestamp => Time.now.to_i,
-        :environment => config.environment,
+        :environment => environment,
         :level => level,
         :language => 'ruby',
         :framework => config.framework,

data/lib/rollbar/configuration.rb

@@ -41,7 +41,7 @@ module Rollbar
       @default_logger = lambda { Logger.new(STDERR) }
       @enabled = nil  # set to true when configure is called
       @endpoint = DEFAULT_ENDPOINT
-      @environment = 'unspecified'
+      @environment = nil
       @exception_level_filters = {
         'ActiveRecord::RecordNotFound' => 'warning',
         'AbstractController::ActionNotFound' => 'warning',

data/lib/rollbar/version.rb

@@ -1,3 +1,3 @@
 module Rollbar
-  VERSION = "0.11.2"
+  VERSION = "0.11.3"
 end

data/spec/controllers/home_controller_spec.rb

@@ -26,6 +26,17 @@ describe HomeController do
       data = Rollbar.send(:base_data)
       data[:environment].should == 'dev'
     end
+    
+    it 'should use the default "unspecified" environment if rails env ends up being empty' do
+      old_env = ::Rails.env
+      ::Rails.env = ''
+      Rollbar::Rails.initialize
+      
+      data = Rollbar.send(:base_data)
+      data[:environment].should == 'unspecified'
+      
+      ::Rails.env = old_env
+    end
   end
 
   context "rollbar controller methods with %s requests" % (local? ? 'local' : 'non-local') do

metadata

@@ -1,104 +1,127 @@
 --- !ruby/object:Gem::Specification
 name: rollbar
 version: !ruby/object:Gem::Version
-  version: 0.11.2
-  prerelease: 
+  version: 0.11.3
 platform: ruby
 authors:
 - Brian Rue
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-10-08 00:00:00.000000000 Z
+date: 2013-10-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: multi_json
-  requirement: &4457060 !ruby/object:Gem::Requirement
-    none: false
+  requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
         version: '1.5'
   type: :runtime
   prerelease: false
-  version_requirements: *4457060
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '1.5'
 - !ruby/object:Gem::Dependency
   name: rails
-  requirement: &4456180 !ruby/object:Gem::Requirement
-    none: false
+  requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
         version: 3.0.0
   type: :development
   prerelease: false
-  version_requirements: *4456180
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: 3.0.0
 - !ruby/object:Gem::Dependency
   name: rspec-rails
-  requirement: &4455300 !ruby/object:Gem::Requirement
-    none: false
+  requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
         version: 2.12.0
   type: :development
   prerelease: false
-  version_requirements: *4455300
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 2.12.0
 - !ruby/object:Gem::Dependency
   name: database_cleaner
-  requirement: &4454140 !ruby/object:Gem::Requirement
-    none: false
+  requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
         version: 1.0.0
   type: :development
   prerelease: false
-  version_requirements: *4454140
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 1.0.0
 - !ruby/object:Gem::Dependency
   name: girl_friday
-  requirement: &4452780 !ruby/object:Gem::Requirement
-    none: false
+  requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
         version: 0.11.1
   type: :development
   prerelease: false
-  version_requirements: *4452780
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: 0.11.1
 - !ruby/object:Gem::Dependency
   name: sucker_punch
-  requirement: &4451980 !ruby/object:Gem::Requirement
-    none: false
+  requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
         version: 1.0.0
   type: :development
   prerelease: false
-  version_requirements: *4451980
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: 1.0.0
 - !ruby/object:Gem::Dependency
   name: sidekiq
-  requirement: &4451200 !ruby/object:Gem::Requirement
-    none: false
+  requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
         version: 2.13.0
   type: :development
   prerelease: false
-  version_requirements: *4451200
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: 2.13.0
 - !ruby/object:Gem::Dependency
   name: genspec
-  requirement: &4450460 !ruby/object:Gem::Requirement
-    none: false
+  requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
         version: 0.2.7
   type: :development
   prerelease: false
-  version_requirements: *4450460
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: 0.2.7
 description: Rails plugin to catch and send exceptions to Rollbar
 email:
 - brian@rollbar.com
@@ -204,27 +227,26 @@ files:
 homepage: https://github.com/rollbar/rollbar-gem
 licenses:
 - MIT
+metadata: {}
 post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
-  - - ! '>='
+  - - '>='
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
-  - - ! '>='
+  - - '>='
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 1.8.11
+rubygems_version: 2.0.3
 signing_key: 
-specification_version: 3
+specification_version: 4
 summary: Reports exceptions to Rollbar
 test_files:
 - spec/controllers/home_controller_spec.rb