Change to only logging when unable to parse user agent

rubygems · Feb 21, 2024 · 0c48ef3 · 0c48ef3
1 parent ba422e5
commit 0c48ef3
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/app/models/events/recordable.rb b/app/models/events/recordable.rb
@@ -9,8 +9,8 @@ def record_event!(tag, request: Current.request, **additional)
       begin
         user_agent_info = Gemcutter::UserAgentParser.call(user_agent)
         additional[:user_agent_info] = user_agent_info
-      rescue Gemcutter::UserAgentParser::UnableToParse => e
-        Rails.error.report(e, context: { user_agent: }, handled: true)
+      rescue Gemcutter::UserAgentParser::UnableToParse
+        nil
       end
     end
 

diff --git a/lib/gemcutter/user_agent_parser.rb b/lib/gemcutter/user_agent_parser.rb
@@ -33,6 +33,8 @@ def call(user_agent, exclusive: false)
 
       return ret if ret
 
+      logger.warn("No parser could parse the user agent", user_agent: user_agent)
+
       raise UnableToParse, "No parser could parse the user agent"
     end
   end