guard against nil values

rumblelabs · Jan 7, 2014 · c2b5e12 · c2b5e12
1 parent 41505cf
commit c2b5e12
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/lib/rails_exception_handler/parser.rb b/lib/rails_exception_handler/parser.rb
@@ -14,8 +14,8 @@ def initialize(env, request, exception, controller)
   def parse
     @internal_info[:error_class] =    @exception.class.to_s
     @internal_info[:target_url] =     @request.url
-    @internal_info[:referer_url] =    @request.referer || ""
-    @internal_info[:user_agent] =     @request.user_agent || ""
+    @internal_info[:referer_url] =    @request.referer
+    @internal_info[:user_agent] =     @request.user_agent
 
     config = RailsExceptionHandler.configuration
     config.request_info_block.call(@external_info, @request) if(config.request_info_block)
@@ -75,17 +75,17 @@ def filter_anon_404s
   end
 
   def filter_user_agent_regxp(regxp)
-    result = @internal_info[:user_agent].match(regxp)
+    result = (@internal_info[:user_agent] || "").match(regxp)
     result != nil
   end
 
   def filter_target_url_regxp(regxp)
-    result = @internal_info[:target_url].match(regxp)
+    result = (@internal_info[:target_url] || "").match(regxp)
     result != nil
   end
 
   def filter_referer_url_regxp(regxp)
-    result = @internal_info[:referer_url].match(regxp)
+    result = (@internal_info[:referer_url] || "").match(regxp)
     result != nil
   end
 end