diff --git a/actix-web/CHANGES.md b/actix-web/CHANGES.md index fbe812259..9d09bacbe 100644 --- a/actix-web/CHANGES.md +++ b/actix-web/CHANGES.md @@ -6,6 +6,7 @@ - Improve handling of non-UTF-8 header values in `Logger` middleware. - Add `HttpServer::shutdown_signal()` method. - Mark `HttpServer` as `#[must_use]`. +- Allow SVG images to be compressed by the `Compress` middleware. - Re-export `mime` dependency. - Update `brotli` dependency to `8`. @@ -28,7 +29,6 @@ - On Windows, an error is now returned from `HttpServer::bind()` (or TLS variants) when binding to a socket that's already in use. - Update `brotli` dependency to `7`. - Minimum supported Rust version (MSRV) is now 1.75. -- Fix Svg images not being compressed by the `Compress` middleware. ## 4.9.0 diff --git a/actix-web/src/middleware/compress.rs b/actix-web/src/middleware/compress.rs index fccfaffee..7f0d8a4fb 100644 --- a/actix-web/src/middleware/compress.rs +++ b/actix-web/src/middleware/compress.rs @@ -191,13 +191,10 @@ where None => true, Some(hdr) => { match hdr.to_str().ok().and_then(|hdr| hdr.parse::().ok()) { - Some(mime) - if mime.type_().as_str() == "image" - && mime.subtype().as_str() != "svg+xml" => - { - false + Some(mime) if mime.type_() == mime::IMAGE => { + matches!(mime.subtype(), mime::SVG) } - Some(mime) if mime.type_().as_str() == "video" => false, + Some(mime) if mime.type_() == mime::VIDEO => false, _ => true, } }