Bug 10935 - Binary artwork shown in scanner.log (FLAC, _preCheckAttributes), making the system BEEP all the time
: Binary artwork shown in scanner.log (FLAC, _preCheckAttributes), making the s...
Status: RESOLVED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Scanner
: 7.4.0
: PC Windows XP
: -- normal with 1 vote (vote)
: ---
Assigned To: Unassigned bug - please assign me!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-02-02 15:43 UTC by Moonbase
Modified: 2009-02-03 01:12 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 Moonbase 2009-02-02 15:43:11 UTC
SC7.4-24847/Windows (Perl version)

Slim::Schema::_preCheckAttributes is displaying binary content of ARTWORK into the log. When debug is on, this leads to awful bios PC speaker beeping (if using the Perl version from a commandline) -- and unneccessary delay.

Log view is also messed up by that: When viewing the log in a browser, it often suddenly suggests downloading scanner.log (instead of just displaying it).

I suggest disabling ARTWORK output (it will also unneccessarily create HUGE log files if debug is on).
Comment 1 Moonbase 2009-02-02 16:06:50 UTC
Index: Schema.pm
===================================================================
--- Schema.pm	(revision 24845)
+++ Schema.pm	(working copy)
@@ -1998,7 +1998,12 @@
 
 		while (my ($tag, $value) = each %{$attributes}) {
 
-			$log->debug(".. $tag : $value") if defined $value;
+			# Bug 10935 - Also need to avoid artwork dump here (see below)
+			if ($tag eq 'ARTWORK') {
+				$log->debug(".. $tag : [Binary Image Data]") if defined $value;
+			} else {
+				$log->debug(".. $tag : $value") if defined $value;
+			}
 		}
 
 		$log->debug("* Deferred Attributes *");
Comment 2 Michael Herger 2009-02-03 01:12:46 UTC
change 24850