Merge pull request rails#50359 from flavorjones/serve-svg-with-compression_rebase

feature: Allow serving compressed SVG images (rebased)

Author: byroot

actionpack/CHANGELOG.md

@@ -1,3 +1,7 @@
+*   Add `image/svg+xml` to the compressible content types of ActionDispatch::Static
+
+    *Georg Ledermann*
+
 *   Add instrumentation for ActionController::Live#send_stream
 
     Allows subscribing to `send_stream` events. The event payload contains the filename, disposition, and type.

actionpack/lib/action_dispatch/middleware/static.rb

@@ -50,7 +50,7 @@ class FileHandler
       "identity" => nil
     }
 
-    def initialize(root, index: "index", headers: {}, precompressed: %i[ br gzip ], compressible_content_types: /\A(?:text\/|application\/javascript)/)
+    def initialize(root, index: "index", headers: {}, precompressed: %i[ br gzip ], compressible_content_types: /\A(?:text\/|application\/javascript|image\/svg\+xml)/)
       @root = root.chomp("/").b
       @index = index
 

actionpack/test/dispatch/static_test.rb

@@ -175,6 +175,15 @@ def test_serves_gzip_files_when_header_set
     assert_not_equal "gzip", response.headers["content-encoding"]
   end
 
+  def test_serves_gzip_files_when_svg
+    file_name = "/gzip/logo-bcb6d75d927347158af5.svg"
+    response  = get(file_name, "HTTP_ACCEPT_ENCODING" => "gzip")
+    assert_gzip  file_name, response
+    assert_equal "image/svg+xml", response.headers["Content-Type"]
+    assert_equal "accept-encoding",        response.headers["Vary"]
+    assert_equal "gzip",                   response.headers["Content-Encoding"]
+  end
+
   def test_set_vary_when_origin_compressed_but_client_cant_accept
     file_name = "/gzip/application-a71b3024f80aea3181c09774ca17e712.js"
     response  = get(file_name, "HTTP_ACCEPT_ENCODING" => "None")