Bug 4369 - songinfo download returns 404
: songinfo download returns 404
Status: RESOLVED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Web Interface
: 6.5.0
: Macintosh Other
: P2 normal (vote)
: ---
Assigned To: Chris Owens
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2006-10-13 23:21 UTC by KDF
Modified: 2006-10-13 23:49 UTC (History)
0 users

See Also:
Category: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description KDF 2006-10-13 23:21:09 UTC
the patch matching isn't right, as the url has a trailing slash, where the regex looks for end of lin on "download".

fix is:

Index: Slim/Web/HTTP.pm
===================================================================
--- Slim/Web/HTTP.pm    (revision 10335)
+++ Slim/Web/HTTP.pm    (working copy)
@@ -878,7 +878,7 @@
 
           ($body, $mtime, $inode, $size, $contentType) = Slim::Web::Graphics::processCoverArtRequest($client, $path);
 
-       } elsif ($path =~ /music\/(\d+)\/download$/) {
+       } elsif ($path =~ /music\/(\d+)\/download/) {
 
           my $obj = Slim::Schema->find('Track', $1);
Comment 1 KDF 2006-10-13 23:49:04 UTC
committed to trunk at change 10336, and 6.5.1 at change 10337.