demarches-normaliennes/app/jobs/virus_scanner_job.rb

class VirusScannerJob < ApplicationJob
  queue_as :active_storage_analysis

  discard_on ActiveRecord::RecordNotFound

  def perform(blob)
    metadata = extract_metadata_via_virus_scanner(blob)
    blob.update!(metadata: blob.metadata.merge(metadata))
  end

  def extract_metadata_via_virus_scanner(blob)
    ActiveStorage::VirusScanner.new(blob).metadata
  end
end
Save virus scan status to blob metadata 2019-05-02 11:37:35 +02:00			`class VirusScannerJob < ApplicationJob`
config: flip config.active_storage.queues ActiveStorage jobs are now moved to their own queue. For consistency, we also move our own analysis jobs (VirusScannerJob) on the same `:active_storage_analysis` queue. 2020-07-13 17:48:09 +02:00			`queue_as :active_storage_analysis`

jobs: ignore missing blob during virus scan We currently have many failed VirusScannerJob enqueued, because the underlying blob is missing. This PR fixes the issue by discarding the job in those cases (because if the blob is gone, the job is never going to succeed). The implementation is based on a similar issue encoutered by the ActiveStorage::AnalyzeJob. See https://github.com/rails/rails/commit/06f8baf73c677303353c79609838caaeec3ccc9a 2020-07-13 14:31:21 +02:00			`discard_on ActiveRecord::RecordNotFound`

Save virus scan status to blob metadata 2019-05-02 11:37:35 +02:00			`def perform(blob)`
			`metadata = extract_metadata_via_virus_scanner(blob)`
			`blob.update!(metadata: blob.metadata.merge(metadata))`
			`end`

			`def extract_metadata_via_virus_scanner(blob)`
			`ActiveStorage::VirusScanner.new(blob).metadata`
			`end`
			`end`