feat: Save error backtrace for sync errors for better debugging (#1578)

Co-authored-by: Tony Vincent Yesudas <[email protected]>
maybe-finance · Dec 30, 2024 · 9d217af · 9d217af
1 parent 77def1d
commit 9d217af
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 2 deletions.
diff --git a/app/models/sync.rb b/app/models/sync.rb
@@ -34,6 +34,11 @@ def fail!(error)
         scope.set_context("sync", { id: id })
       end
 
-      update! status: :failed, error: error.message, last_ran_at: Time.current
+      update!(
+        status: :failed,
+        error: error.message,
+        error_backtrace: error.backtrace&.first(10),
+        last_ran_at: Time.current
+      )
     end
 end
diff --git a/db/migrate/20241227142333_add_error_trace_to_syncs.rb b/db/migrate/20241227142333_add_error_trace_to_syncs.rb
@@ -0,0 +1,5 @@
+class AddErrorTraceToSyncs < ActiveRecord::Migration[7.2]
+  def change
+    add_column :syncs, :error_backtrace, :text, array: true
+  end
+end
diff --git a/db/schema.rb b/db/schema.rb