ratonvirus 0.4.2 → 0.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bf996f1b6f63fe570fb46f62c91625c0e8ad3ec073f595802d580c399c1fca27
-  data.tar.gz: 6ad46dd7435948e0750b00cb78820f550b1edd8229923ed84e118ac09feacbde
+  metadata.gz: ad1408747188a58c2a0d1ad7fea36a4a2144daf86c3f0ab82e568beb0eb78335
+  data.tar.gz: 4d47b462f78bef2abfc611780ce85972e63ab35318c8492146f5b87aec4ad507
 SHA512:
-  metadata.gz: 3eacdeb037efa1a36dc2c7c600d39cd29c52ca23abebe2f28b054046de9e73553dcb7b53b1907f2db6075df7af0cb85b9be2c869dcf3c0a8ed6dc747580caa27
-  data.tar.gz: 47e950609423daec9177c450bcf96d7dab1273ec288ecf4becc8f881f04121207487b05271bd035c2fe8036d1f198f6a0202e2d6a5e458bcd9d2338ac21933e7
+  metadata.gz: 717b88a5ba971f64de16d81a6af5e4bb82a4832fce8780801c63eaf64aa231dcb1fdbb549efa40a30d64661f18c31c6f1b5c80af7bd93f768f75b126a30fde2f
+  data.tar.gz: 1195298ca2136fad104b00b0eb56ed9f1552c2cd6122cc08487d93cbf82793de188b3c31239a56e32ae6f86b1006be96fd8c0e1365b1c282b3d2078c356f97b7

data/CHANGELOG.md

@@ -1,3 +1,20 @@
+# v0.5.0
+
+Changed:
+
+- Scanning is now skipped for blobs that have a related attachment that has been persisted in order to avoid scanning the same file multiple times. We assume that the file needs to be scanned only once during the first time it is being attached to a record because otherwise it would be re-scanned every time when the attached record is updated (in which case the file would not be changed, as if the file is changed, also the attached blob is changed).
+
+Fixed:
+
+- Skip antivirus scan for persisted blobs [#39](https://github.com/mainio/ratonvirus/pull/39)
+- XAmzContentSHA256Mismatch when using active storage with s3 backend [#36](https://github.com/mainio/ratonvirus/issues/36)
+
+# v0.4.3
+
+Fixed:
+
+- Manual scan is not working for File objects [#32](https://github.com/mainio/ratonvirus/pull/32)
+
 # v0.4.2
 
 Fixed:

data/README.md

@@ -250,6 +250,26 @@ For installing ClamAV, refer to
 For further information about the configurations and how to create custom
 scanners, please refer to the [documentation](docs/index.md).
 
+## Common issues
+
+### NoMethodError (undefined method `attached?' ...)
+
+Multiple issues have been opened because of users getting the following error
+when trying to run a scan against a file path:
+
+```
+NoMethodError (undefined method `attached?' for "/path/to/file.pdf":String)
+```
+
+Most of the times this issue is because you have not configured the storage
+correctly. If you want to run direct scans against files, please use the
+`:filepath` storage option as described at
+[Manually checking for viruses](#manually-checking-for-viruses).
+
+If you need both: scans against file paths and scans against Active Storage
+attached files, please use the `:multi` storage option as described in the same
+section of the documentation.
+
 ## License
 
 MIT, see [LICENSE](LICENSE).

data/lib/ratonvirus/storage/active_storage.rb

@@ -95,7 +95,7 @@ module Ratonvirus
       def yield_processable_from(change, &_block)
         attachable = change.attachable
         return unless attachable
-        return if attachable.is_a?(::ActiveStorage::Blob)
+        return if attachable.is_a?(::ActiveStorage::Blob) && change.attachment.persisted?
 
         # If the attachable is a string, it is a reference to an already
         # existing blob. This can happen e.g. when the file blob is uploaded

data/lib/ratonvirus/storage/filepath.rb

@@ -24,7 +24,7 @@ module Ratonvirus
         return unless block_given?
 
         return unless asset
-        return if asset.empty?
+        return if asset.respond_to?(:empty?) && asset.empty?
 
         if asset.respond_to?(:path)
           # A file asset that responds to path (e.g. default `File`
@@ -33,7 +33,6 @@ module Ratonvirus
 
           return
         end
-
         # Plain file path string provided as resource
         yield asset
       end

data/lib/ratonvirus/storage/support/io_handling.rb

@@ -23,8 +23,13 @@ module Ratonvirus
           begin
             tempfile.binmode
             if io.is_a?(StringIO)
+              orig_pos = io.pos
+              io.rewind
+
               # cannot specify src_offset for non-IO
               IO.copy_stream(io, tempfile)
+
+              io.pos = orig_pos
             else
               IO.copy_stream(io, tempfile, nil, 0)
             end

data/lib/ratonvirus/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Ratonvirus
-  VERSION = "0.4.2"
+  VERSION = "0.5.0"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ratonvirus
 version: !ruby/object:Gem::Version
-  version: 0.4.2
+  version: 0.5.0
 platform: ruby
 authors:
 - Antti Hukkanen
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-03-25 00:00:00.000000000 Z
+date: 2026-03-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport