aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorGeorge Claghorn <george@basecamp.com>2018-07-11 22:04:42 -0400
committerGeorge Claghorn <george@basecamp.com>2018-07-11 22:04:42 -0400
commita1735a0dde6f6efa4cda6347da71a056740da1d3 (patch)
tree11c9343ac76a2d98c70d48c3e8aec12e72a1b675
parentf16fdefe3243978aef11204b12c25b86c022ca7a (diff)
downloadrails-a1735a0dde6f6efa4cda6347da71a056740da1d3.tar.gz
rails-a1735a0dde6f6efa4cda6347da71a056740da1d3.tar.bz2
rails-a1735a0dde6f6efa4cda6347da71a056740da1d3.zip
Remove overly-broad retry in ActiveStorage::PurgeJob
We've never intentionally tripped this retry in production. Add retries for specific errors as needed.
-rw-r--r--activestorage/app/jobs/active_storage/purge_job.rb3
1 files changed, 0 insertions, 3 deletions
diff --git a/activestorage/app/jobs/active_storage/purge_job.rb b/activestorage/app/jobs/active_storage/purge_job.rb
index 98874d2250..95ae7d0c30 100644
--- a/activestorage/app/jobs/active_storage/purge_job.rb
+++ b/activestorage/app/jobs/active_storage/purge_job.rb
@@ -2,9 +2,6 @@
# Provides asynchronous purging of ActiveStorage::Blob records via ActiveStorage::Blob#purge_later.
class ActiveStorage::PurgeJob < ActiveStorage::BaseJob
- # FIXME: Limit this to a custom ActiveStorage error
- retry_on StandardError
-
def perform(blob)
blob.purge
end