Use Marcel::TYPE_EXTS instead of Marcel::TYPES

carrierwaveuploader · Feb 29, 2024 · 2408ed4 · 2408ed4
1 parent ed87991
commit 2408ed4
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/carrierwave.gemspec b/carrierwave.gemspec
@@ -23,7 +23,7 @@ Gem::Specification.new do |s|
   s.add_dependency "activesupport", ">= 6.0.0"
   s.add_dependency "activemodel", ">= 6.0.0"
   s.add_dependency "image_processing", "~> 1.1"
-  s.add_dependency "marcel", "~> 1.0.0"
+  s.add_dependency "marcel", "~> 1.0.3"
   s.add_dependency "addressable", "~> 2.6"
   s.add_dependency "ssrf_filter", "~> 1.0"
   s.add_development_dependency "csv", "~> 3.0"

diff --git a/lib/carrierwave/downloader/remote_file.rb b/lib/carrierwave/downloader/remote_file.rb
@@ -33,9 +33,8 @@ def headers
 
       def original_filename
         filename = filename_from_header || filename_from_uri
-        mime_type = Marcel::TYPES[content_type]
-        unless File.extname(filename).present? || mime_type.blank?
-          extension = mime_type[0].first
+        extension = Marcel::TYPE_EXTS[content_type]
+        unless File.extname(filename).present? || extension.blank?
           filename = "#{filename}.#{extension}"
         end
         filename