From cba2d02990bd1b6c4f162cf522c5190e914fd5c5 Mon Sep 17 00:00:00 2001 From: Pete Matsyburka Date: Wed, 3 Dec 2025 13:20:52 +0200 Subject: [PATCH] skip webhook not found --- app/jobs/process_submission_expired_job.rb | 4 +++- app/jobs/send_form_completed_webhook_request_job.rb | 9 +++++++-- app/jobs/send_form_declined_webhook_request_job.rb | 9 +++++++-- app/jobs/send_form_started_webhook_request_job.rb | 9 +++++++-- app/jobs/send_form_viewed_webhook_request_job.rb | 9 +++++++-- app/jobs/send_submission_archived_webhook_request_job.rb | 9 +++++++-- .../send_submission_completed_webhook_request_job.rb | 9 +++++++-- app/jobs/send_submission_created_webhook_request_job.rb | 9 +++++++-- app/jobs/send_submission_expired_webhook_request_job.rb | 9 +++++++-- app/jobs/send_template_created_webhook_request_job.rb | 9 +++++++-- app/jobs/send_template_updated_webhook_request_job.rb | 9 +++++++-- app/jobs/send_test_webhook_request_job.rb | 9 ++++++--- 12 files changed, 79 insertions(+), 24 deletions(-) diff --git a/app/jobs/process_submission_expired_job.rb b/app/jobs/process_submission_expired_job.rb index fe0f92d1..d8303568 100644 --- a/app/jobs/process_submission_expired_job.rb +++ b/app/jobs/process_submission_expired_job.rb @@ -4,7 +4,9 @@ class ProcessSubmissionExpiredJob include Sidekiq::Job def perform(params = {}) - submission = Submission.find(params['submission_id']) + submission = Submission.find_by(id: params['submission_id']) + + return unless submission return if submission.archived_at? return if submission.template&.archived_at? diff --git a/app/jobs/send_form_completed_webhook_request_job.rb b/app/jobs/send_form_completed_webhook_request_job.rb index e4d23795..e025bb4c 100644 --- a/app/jobs/send_form_completed_webhook_request_job.rb +++ b/app/jobs/send_form_completed_webhook_request_job.rb @@ -8,8 +8,13 @@ class SendFormCompletedWebhookRequestJob MAX_ATTEMPTS = 12 def perform(params = {}) - submitter = Submitter.find(params['submitter_id']) - webhook_url = WebhookUrl.find(params['webhook_url_id']) + submitter = Submitter.find_by(id: params['submitter_id']) + + return unless submitter + + webhook_url = WebhookUrl.find_by(id: params['webhook_url_id']) + + return unless webhook_url attempt = params['attempt'].to_i diff --git a/app/jobs/send_form_declined_webhook_request_job.rb b/app/jobs/send_form_declined_webhook_request_job.rb index fa0f537e..32d63bde 100644 --- a/app/jobs/send_form_declined_webhook_request_job.rb +++ b/app/jobs/send_form_declined_webhook_request_job.rb @@ -8,8 +8,13 @@ class SendFormDeclinedWebhookRequestJob MAX_ATTEMPTS = 10 def perform(params = {}) - submitter = Submitter.find(params['submitter_id']) - webhook_url = WebhookUrl.find(params['webhook_url_id']) + submitter = Submitter.find_by(id: params['submitter_id']) + + return unless submitter + + webhook_url = WebhookUrl.find_by(id: params['webhook_url_id']) + + return unless webhook_url attempt = params['attempt'].to_i diff --git a/app/jobs/send_form_started_webhook_request_job.rb b/app/jobs/send_form_started_webhook_request_job.rb index c233f682..cc278d2a 100644 --- a/app/jobs/send_form_started_webhook_request_job.rb +++ b/app/jobs/send_form_started_webhook_request_job.rb @@ -8,8 +8,13 @@ class SendFormStartedWebhookRequestJob MAX_ATTEMPTS = 10 def perform(params = {}) - submitter = Submitter.find(params['submitter_id']) - webhook_url = WebhookUrl.find(params['webhook_url_id']) + submitter = Submitter.find_by(id: params['submitter_id']) + + return unless submitter + + webhook_url = WebhookUrl.find_by(id: params['webhook_url_id']) + + return unless webhook_url attempt = params['attempt'].to_i diff --git a/app/jobs/send_form_viewed_webhook_request_job.rb b/app/jobs/send_form_viewed_webhook_request_job.rb index 11c891a3..8f6f0d6d 100644 --- a/app/jobs/send_form_viewed_webhook_request_job.rb +++ b/app/jobs/send_form_viewed_webhook_request_job.rb @@ -8,8 +8,13 @@ class SendFormViewedWebhookRequestJob MAX_ATTEMPTS = 10 def perform(params = {}) - submitter = Submitter.find(params['submitter_id']) - webhook_url = WebhookUrl.find(params['webhook_url_id']) + submitter = Submitter.find_by(id: params['submitter_id']) + + return unless submitter + + webhook_url = WebhookUrl.find_by(id: params['webhook_url_id']) + + return unless webhook_url attempt = params['attempt'].to_i diff --git a/app/jobs/send_submission_archived_webhook_request_job.rb b/app/jobs/send_submission_archived_webhook_request_job.rb index f8ee50fc..77ba6d57 100644 --- a/app/jobs/send_submission_archived_webhook_request_job.rb +++ b/app/jobs/send_submission_archived_webhook_request_job.rb @@ -8,8 +8,13 @@ class SendSubmissionArchivedWebhookRequestJob MAX_ATTEMPTS = 10 def perform(params = {}) - submission = Submission.find(params['submission_id']) - webhook_url = WebhookUrl.find(params['webhook_url_id']) + submission = Submission.find_by(id: params['submission_id']) + + return unless submission + + webhook_url = WebhookUrl.find_by(id: params['webhook_url_id']) + + return unless webhook_url attempt = params['attempt'].to_i diff --git a/app/jobs/send_submission_completed_webhook_request_job.rb b/app/jobs/send_submission_completed_webhook_request_job.rb index 41ec1d64..3832eb35 100644 --- a/app/jobs/send_submission_completed_webhook_request_job.rb +++ b/app/jobs/send_submission_completed_webhook_request_job.rb @@ -8,8 +8,13 @@ class SendSubmissionCompletedWebhookRequestJob MAX_ATTEMPTS = 10 def perform(params = {}) - submission = Submission.find(params['submission_id']) - webhook_url = WebhookUrl.find(params['webhook_url_id']) + submission = Submission.find_by(id: params['submission_id']) + + return unless submission + + webhook_url = WebhookUrl.find_by(id: params['webhook_url_id']) + + return unless webhook_url attempt = params['attempt'].to_i diff --git a/app/jobs/send_submission_created_webhook_request_job.rb b/app/jobs/send_submission_created_webhook_request_job.rb index 5c09311a..c9fb1ff6 100644 --- a/app/jobs/send_submission_created_webhook_request_job.rb +++ b/app/jobs/send_submission_created_webhook_request_job.rb @@ -8,8 +8,13 @@ class SendSubmissionCreatedWebhookRequestJob MAX_ATTEMPTS = 10 def perform(params = {}) - submission = Submission.find(params['submission_id']) - webhook_url = WebhookUrl.find(params['webhook_url_id']) + submission = Submission.find_by(id: params['submission_id']) + + return unless submission + + webhook_url = WebhookUrl.find_by(id: params['webhook_url_id']) + + return unless webhook_url attempt = params['attempt'].to_i diff --git a/app/jobs/send_submission_expired_webhook_request_job.rb b/app/jobs/send_submission_expired_webhook_request_job.rb index 288f3ac5..d89b4f3a 100644 --- a/app/jobs/send_submission_expired_webhook_request_job.rb +++ b/app/jobs/send_submission_expired_webhook_request_job.rb @@ -8,8 +8,13 @@ class SendSubmissionExpiredWebhookRequestJob MAX_ATTEMPTS = 10 def perform(params = {}) - submission = Submission.find(params['submission_id']) - webhook_url = WebhookUrl.find(params['webhook_url_id']) + submission = Submission.find_by(id: params['submission_id']) + + return unless submission + + webhook_url = WebhookUrl.find_by(id: params['webhook_url_id']) + + return unless webhook_url attempt = params['attempt'].to_i diff --git a/app/jobs/send_template_created_webhook_request_job.rb b/app/jobs/send_template_created_webhook_request_job.rb index cb2d6e5d..cb0af29f 100644 --- a/app/jobs/send_template_created_webhook_request_job.rb +++ b/app/jobs/send_template_created_webhook_request_job.rb @@ -8,8 +8,13 @@ class SendTemplateCreatedWebhookRequestJob MAX_ATTEMPTS = 10 def perform(params = {}) - template = Template.find(params['template_id']) - webhook_url = WebhookUrl.find(params['webhook_url_id']) + template = Template.find_by(id: params['template_id']) + + return unless template + + webhook_url = WebhookUrl.find_by(id: params['webhook_url_id']) + + return unless webhook_url attempt = params['attempt'].to_i diff --git a/app/jobs/send_template_updated_webhook_request_job.rb b/app/jobs/send_template_updated_webhook_request_job.rb index 7ef3c22b..46a67dcc 100644 --- a/app/jobs/send_template_updated_webhook_request_job.rb +++ b/app/jobs/send_template_updated_webhook_request_job.rb @@ -8,8 +8,13 @@ class SendTemplateUpdatedWebhookRequestJob MAX_ATTEMPTS = 10 def perform(params = {}) - template = Template.find(params['template_id']) - webhook_url = WebhookUrl.find(params['webhook_url_id']) + template = Template.find_by(id: params['template_id']) + + return unless template + + webhook_url = WebhookUrl.find_by(id: params['webhook_url_id']) + + return unless webhook_url attempt = params['attempt'].to_i diff --git a/app/jobs/send_test_webhook_request_job.rb b/app/jobs/send_test_webhook_request_job.rb index 3caf5607..682a3aac 100644 --- a/app/jobs/send_test_webhook_request_job.rb +++ b/app/jobs/send_test_webhook_request_job.rb @@ -8,10 +8,13 @@ class SendTestWebhookRequestJob USER_AGENT = 'DocuSeal.com Webhook' def perform(params = {}) - submitter = Submitter.find(params['submitter_id']) - webhook_url = WebhookUrl.find(params['webhook_url_id']) + submitter = Submitter.find_by(id: params['submitter_id']) - return unless webhook_url && submitter + return unless submitter + + webhook_url = WebhookUrl.find_by(id: params['webhook_url_id']) + + return unless webhook_url Faraday.post(webhook_url.url, {