Bug 9188 – MusicIP should stop using stream handler as generic HTTP mechanism

Bug 9188 - MusicIP should stop using stream handler as generic HTTP mechanism

Summary:

MusicIP should stop using stream handler as generic HTTP mechanism

Status:	CLOSED FIXED

Product:	Logitech Media Server
Classification:	Unclassified
Component:	Plugin
Version:	7.1
Platform:	PC Other

Importance:	P3 normal (vote)
Target Milestone:	7.x
Assigned To:	Michael Herger

URL:
Keywords:

Depends on:
Blocks:	8766
	Show dependency tree / graph

Reported:	2008-08-16 23:12 UTC by Alan Young
Modified:	2009-07-31 10:27 UTC (History)
CC List:	1 user (show)

See Also:
Category:	---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Alan Young 2008-08-16 23:12:03 UTC

MusicMagic abuses the Slim::Player::Protocols::HTTP stream protocol handler as a generic HTTP fetch mechanism. It should stop this and use a more suitable mechanism, either AsyncHTTP or standard sync http from CPAN.

Comment 1 KDF 2008-08-16 23:18:48 UTC

much of this was being covered by bug 8277, but there was a limit to what was desired/possible with SimpleAsync

Comment 2 Michael Herger 2008-08-17 08:48:04 UTC

It does indeed use LWP::Simple in some places already. Might do this in all places. This might also fix the issue where we don't get any answer from our HTTP handler (stale filters issue).

Comment 3 Michael Herger 2008-08-18 03:28:38 UTC

change 22728 - use LWP::UserAgent instead of Slim::Player::Protocols::HTTP

Might make grabMoods async, as grabFilters, updating regularly?

Comment 4 James Richardson 2008-12-15 12:06:13 UTC

This bug has been fixed in the 7.3.0 release version of SqueezeCenter!

Please download the new version from http://www.slimdevices.com/su_downloads.html if you haven't already.  

If you are still experiencing this problem, feel free to reopen the bug with your new comments and we'll have another look.

Comment 5 Chris Owens 2009-07-31 10:27:27 UTC

Reduce number of active targets for SC