Allow serving compressed SVG images

actionpack/CHANGELOG.md

@@ -1,3 +1,7 @@
+*   Add `image/svg+xml` to the compressible content types of ActionDispatch::Static
+
+    *Georg Ledermann*
+
 *   Allow permitting numeric params.
 
     Previously it was impossible to permit different fields on numeric parameters.

actionpack/lib/action_dispatch/middleware/static.rb

@@ -47,7 +47,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

@@ -167,6 +167,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")