Fix ActiveStorage type detection for email files

Fixes #555
cloudinary · Aug 18, 2024 · ab52429 · ab52429
1 parent a634428
commit ab52429
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 2 deletions.
diff --git a/lib/active_storage/service/cloudinary_service.rb b/lib/active_storage/service/cloudinary_service.rb
@@ -255,7 +255,7 @@ def content_type_to_resource_type(content_type)
       case type
       when 'video', 'audio'
         'video'
-      when 'text'
+      when 'text', 'message'
         'raw'
       when 'application'
         case subtype

diff --git a/spec/active_storage/service/cloudinary_service_spec.rb b/spec/active_storage/service/cloudinary_service_spec.rb
@@ -108,7 +108,7 @@
 
     it "should set raw resource_type for raw formats" do
       key = SecureRandom.base58(24)
-      types = %w[text/* application/*]
+      types = %w[text/* application/* message/*]
 
       types.each do |content_type|
         url = @service.url_for_direct_upload(key, content_type: content_type)

diff --git a/spec/spec_helper.rb b/spec/spec_helper.rb
@@ -1,5 +1,6 @@
 SUFFIX = ENV['TRAVIS_JOB_ID'] || rand(999999999).to_s
 
+require 'bundler/setup'
 require 'rspec'
 require 'rspec/retry'
 require 'rexml/parsers/ultralightparser'