fix: Mute analytics processor on failure (#57)

* Add logger

* Add try catch guard for analytics processor

* Add logger

* Fix silly lint

Author: zachaysan

lib/flagsmith.rb

@@ -92,7 +92,8 @@ def analytics_processor
       @analytics_processor ||=
         Flagsmith::AnalyticsProcessor.new(
           api_client: api_client,
-          timeout: request_timeout_seconds
+          timeout: request_timeout_seconds,
+          logger: @config.logger
         )
     end
 

lib/flagsmith/sdk/analytics_processor.rb

@@ -19,15 +19,20 @@ def initialize(data)
       @analytics_data = {}
       @api_client = data.fetch(:api_client)
       @timeout = data.fetch(:timeout, 3)
+      @logger = data.fetch(:logger)
     end
 
     # Sends all the collected data to the api asynchronously and resets the timer
     def flush
       return if @analytics_data.empty?
 
-      @api_client.post(ENDPOINT, @analytics_data.to_json)
+      begin
+        @api_client.post(ENDPOINT, @analytics_data.to_json)
+        @analytics_data = {}
+      rescue StandardError => e
+        @logger.warn "Temporarily unable to access flag analytics endpoint for exception: #{e}"
+      end
 
-      @analytics_data = {}
       @last_flushed = Time.now
     end
 

spec/sdk/analytics_processor_spec.rb

@@ -14,7 +14,7 @@
   end
 
   subject do Flagsmith::AnalyticsProcessor.new(
-    api_client: flagsmith.api_client, timeout: flagsmith.config.request_timeout_seconds
+    api_client: flagsmith.api_client, timeout: flagsmith.config.request_timeout_seconds, logger: flagsmith.config.logger
     )
   end