Bug 17858 - Web UI shows red X instead of artwork for "local" remote streams
: Web UI shows red X instead of artwork for "local" remote streams
Status: RESOLVED INVALID
Product: Logitech Media Server
Classification: Unclassified
Component: Artwork
: 7.7.1
: PC Windows 7
: P4 normal (vote)
: 7.7.2
Assigned To: Michael Herger
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-01-09 03:33 UTC by AndrewFG
Modified: 2013-05-16 14:01 UTC (History)
0 users

See Also:
Category: ---


Attachments
Web UI Screen Shot (13.93 KB, image/jpeg)
2012-01-09 03:35 UTC, AndrewFG
Details
Fxies for Fix (1.41 KB, patch)
2012-04-05 07:16 UTC, pssc
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Comment 1 AndrewFG 2012-01-09 03:35:02 UTC
Created attachment 7603 [details]
Web UI Screen Shot
Comment 2 Michael Herger 2012-01-10 04:14:36 UTC
We most likely have fixed this in the mobile app - need to backport to SqueezeJS.
Comment 3 SVN Bot 2012-03-06 07:24:26 UTC
 == Auto-comment from SVN commit #33873 to the slim repo by mherger ==
 == http://svn.slimdevices.com/slim?view=revision&revision=33873 ==

Fixed Bug: 17858
Description: not all absolute URLs are publicly available. Don't use imageproxy for local addresses.
Comment 4 Michael Herger 2012-03-06 07:24:47 UTC
Andrew - could you please confirm this is now working as expected? Thanks!
Comment 5 Emmanuel Fusté 2012-03-06 08:10:49 UTC
Not only 172.16.x.x network is private,
but 172.16.0.0/12 (172.16.x.x to 172.31.x.x, RFC1918)
Your fix should be extended to be complete.
Comment 6 SVN Bot 2012-03-07 05:33:39 UTC
 == Auto-comment from SVN commit #33875 to the slim repo by mherger ==
 == http://svn.slimdevices.com/slim?view=revision&revision=33875 ==

Bug: 17858
Description: fix regex to fit private IP addresses.

Thanks for the heads up, Emmanuel!
Comment 7 pssc 2012-04-05 07:16:07 UTC
Created attachment 7660 [details]
Fxies for Fix

This fix breaks the facebook suff as facebooks image URL's are quite rightly over https.... So should we... , fix for images with https ie facebook  screensavers and a complete private networks list...

Patch with fixes...
Comment 8 pssc 2012-04-05 07:30:20 UTC
(In reply to comment #7)
> Created an attachment (id=7660) [details]
> Fxies for Fix
> 
> This fix breaks the facebook suff as facebooks image URL's are quite rightly
> over https.... So should we... , fix for images with https ie facebook 
> screensavers and a complete private networks list...
> 
> Patch with fixes...

Sorry wrong bug but this does break for https urls also.... filed patch against right bug for squeezeplay...
Comment 9 AndrewFG 2012-05-21 17:09:27 UTC
Although this bug is marked as fixed, it seems that there are still edge cases where it does not work.

For example, it works fine in the case where the cover art is served by JRiver Media Center with an URL as follows:

http://192.168.1.33:52100/AArt/96.jpg

BUT, it does NOT works in the case where the cover art is served by Windows Media Player with an URL as follows:

http://192.168.1.33:10243/WMPNSSv4/813663920/0_e0NDRTc1NDlELTI1RjMtNEVFNS04NjFFLTlBQjRCMDRCNjBBNH0uMC40.jpg?albumArt=true

Perhaps LMS still has problems to determine that WMP's long / parameterized URL is a local one??
Comment 10 Michael Herger 2013-05-16 11:08:23 UTC
Andrew - is this still an issue with 7.7.3 nightlies and/or 7.8? If so: what's the exact url being used in the web UI?
Comment 11 AndrewFG 2013-05-16 13:54:36 UTC
(In reply to comment #10)
> Andrew - is this still an issue with 7.7.3 nightlies and/or 7.8? If so:
> what's the exact url being used in the web UI?

Frankly, I wouldn't know. Due to the product phase out, I scaled down my use of LMS to the absolute minimum needed to support my existing devices. And I am no longer interested in helping to extend the envelope or improve the product. Sorry.
Comment 12 Michael Herger 2013-05-16 14:01:24 UTC
Ok. Thanks.