From 9a1545c77ca4baf6495c4643ab0e2746849347df Mon Sep 17 00:00:00 2001
From: Peter Oliver <git@mavit.org.uk>
Date: Sun, 29 Sep 2024 16:25:47 +0100
Subject: [PATCH] Pass SVG images through Slim::Web::ImageProxy without
 resizing.

---
 Slim/Web/ImageProxy.pm | 20 +++++++++++++++++++-
 1 file changed, 19 insertions(+), 1 deletion(-)

diff --git a/Slim/Web/ImageProxy.pm b/Slim/Web/ImageProxy.pm
index f47e091677..1c1e00c861 100644
--- a/Slim/Web/ImageProxy.pm
+++ b/Slim/Web/ImageProxy.pm
@@ -72,6 +72,7 @@ original artwork is of considerable size, where the bandwidth to download the im
 =cut
 
 use strict;
+use File::Slurp qw(read_file);
 use HTTP::Status qw(
 	RC_MOVED_PERMANENTLY
 );
@@ -308,7 +309,7 @@ sub _resizeFromFile {
 		my $cachekey = $item->{cachekey};
 
 		# no need to resize data if we've got it from an external image proxy
-		if ( ($spec =~ /^\.(?:png|jpe?g)/i || $item->{pre_shrunk}) && $http && $http->headers->content_type =~ /image\/(png|jpe?g)/ ) {
+		if ( ($spec =~ /^\.(?:png|jpe?g|svg)/i || $item->{pre_shrunk}) && $http && $http->headers->content_type =~ /image\/(png|jpe?g|svg)/ ) {
 			main::DEBUGLOG && $log->debug("No resizing required - already resized remotely, or original size requested");
 
 			my $ct = $1;
@@ -325,6 +326,22 @@ sub _resizeFromFile {
 
 			$callback && $callback->( $client, $params, $fullpath, @$args );
 		}
+		elsif ( $spec =~ /^\.svg/i and ($http ? $http->headers->content_type =~ /image\/svg\+xml/ : 1) ) {
+			main::DEBUGLOG && $log->debug("No resizing required - scalable vector graphics");
+
+			my $mtime = stat($fullpath)->mtime;
+			my $data = read_file($fullpath, {'binmode' => ':raw'});
+			$cache->set( $cachekey, {
+				content_type  => 'svg',
+				mtime         => $mtime,
+				original_path => $fullpath,
+				data_ref      => \$data,
+			} );
+
+			_setHeaders($args->[1], 'image/svg+xml');
+
+			$callback && $callback->( $client, $params, \$data, @$args );
+		}
 		else {
 			Slim::Utils::ImageResizer->resize($fullpath, $cachekey, $spec, sub {
 				my ($body, $format) = @_;
@@ -364,6 +381,7 @@ sub _setHeaders {
 
 	my $ct = $format =~ /image/ ? $format : "image/$format";
 	$ct =~ s/jpg/jpeg/;
+	$ct .= '+xml' if $ct eq 'image/svg';
 
 	$response->content_type($ct);
 	$response->header( 'Cache-Control' => 'max-age=' . ONE_YEAR );