1043
98%
master: 98%

Ran 27 Jan 2018 01:04AM UTC

Jobs 1

Files 53

Run time 5s

Badge

Embed ▾

pending completion

Build # 1043

Build Type

push

travis-ci

Committed by

no-reply

Commit Message

Add basic virus checking with logging and file deletion

Virus handling is added to `AttachFilesToWorkJob` (which unfortunately must be
copied wholesale from Hyrax).

By default Hyrax embeds virus checking in the model validations, which means a
real `Hydra::PCDM::File` must exist in order to check for a virus and handle
errors. This is somewhat challenging in the case that we want to check
asynchonously and handle errors, since a `File` doesn't exist until an
`ImportJob` is running; in the default actor stack, this is several asynchonous
jobs deep.

By adding an early check, we can avoid creating a `FileSet` for files that
contain a virus, and add error handling earlier in the stack. In this case, we
log to `Rails.logger.error`.

We also clean up the file from disk, which Hyrax does not seem to bother
with(?).

Closes #178.

Run Details

727 of 741 relevant lines covered (98.11%)

69.22 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	1043.1 (2.4.2)	27 Jan 2018 01:04AM UTC	53	98.11	Travis Job 1043.1

Source Files on build 1043

Coverage	∆	File	Lines	Relevant	Covered	Missed	Hits/Line

curationexperts / mahonia / 1043
98%
master: 98%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 1043

curationexperts / mahonia / 1043 98% master: 98%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 1043

curationexperts / mahonia / 1043
98%
master: 98%

README BADGES
x