Browse Source

Do not retry processing ActivityPub jobs raising validation errors (#10614)

* Do not retry processing ActivityPub jobs raising validation errors

Jobs yielding validation errors most probably won't ever be accepted,
so it makes sense not to clutter the queues with retries.

* Lower RecordInvalid error reporting to debug log level

* Remove trailing whitespace
ThibG 5 years ago
parent
commit
b85f216cbc
1 changed files with 2 additions and 0 deletions
  1. 2 0
      app/workers/activitypub/processing_worker.rb

+ 2 - 0
app/workers/activitypub/processing_worker.rb

@@ -7,5 +7,7 @@ class ActivityPub::ProcessingWorker
 
   def perform(account_id, body, delivered_to_account_id = nil)
     ActivityPub::ProcessCollectionService.new.call(body, Account.find(account_id), override_timestamps: true, delivered_to_account_id: delivered_to_account_id, delivery: true)
+  rescue ActiveRecord::RecordInvalid => e
+    Rails.logger.debug "Error processing incoming ActivityPub object: #{e}"
   end
 end