From ae649a5b3fd5d891123324b61dd6d64180e73079 Mon Sep 17 00:00:00 2001
From: Shivam Mishra <scm.mymail@gmail.com>
Date: Thu, 14 Dec 2023 14:03:40 +0530
Subject: [PATCH] refactor: add better logs to Instagram::MessageText (#8538)

---
 app/services/instagram/message_text.rb | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/app/services/instagram/message_text.rb b/app/services/instagram/message_text.rb
index 22114feef..54bff38fa 100644
--- a/app/services/instagram/message_text.rb
+++ b/app/services/instagram/message_text.rb
@@ -16,8 +16,12 @@ class Instagram::MessageText < Instagram::WebhooksBaseService
     inbox_channel(instagram_id)
     # person can connect the channel and then delete the inbox
     return if @inbox.blank?
+
     # This channel might require reauthorization, may be owner might have changed the fb password
-    return if @inbox.channel.reauthorization_required?
+    if @inbox.channel.reauthorization_required?
+      Rails.logger.info("Skipping message processing as reauthorization is required for inbox #{inbox.id}")
+      return
+    end
 
     return unsend_message if message_is_deleted?
 
@@ -36,20 +40,24 @@ class Instagram::MessageText < Instagram::WebhooksBaseService
     end
   end
 
+  # rubocop:disable Metrics/AbcSize
   def ensure_contact(ig_scope_id)
     begin
       k = Koala::Facebook::API.new(@inbox.channel.page_access_token) if @inbox.facebook?
       result = k.get_object(ig_scope_id) || {}
     rescue Koala::Facebook::AuthenticationError => e
       @inbox.channel.authorization_error!
+      Rails.logger.warn("Authorization error for account #{@inbox.account_id} for #{inbox @inbox.id}")
       ChatwootExceptionTracker.new(e, account: @inbox.account).capture_exception
     rescue StandardError, Koala::Facebook::ClientError => e
+      Rails.logger.warn("[FacebookUserFetchClientError]: account_id #{@inbox.account_id} inbox_id #{inbox @inbox.id}")
       Rails.logger.warn("[FacebookUserFetchClientError]: #{e.message}")
       ChatwootExceptionTracker.new(e, account: @inbox.account).capture_exception
     end
 
     find_or_create_contact(result) if defined?(result) && result.present?
   end
+  # rubocop:enable Metrics/AbcSize
 
   def agent_message_via_echo?
     @messaging[:message][:is_echo].present?