Bug 1392 - PodcastBrowser and unknown encoding
: PodcastBrowser and unknown encoding
Status: RESOLVED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Plugin
: 6.1.0
: PC Windows XP
: P3 minor with 1 vote (vote)
: ---
Assigned To: Dan Sully
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2005-04-18 14:34 UTC by Jonas Nordstr�m
Modified: 2009-09-08 09:18 UTC (History)
1 user (show)

See Also:
Category: ---


Attachments
rss feed with windows encoding (7.43 KB, text/xml)
2005-04-21 18:47 UTC, Dave Cohen
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Jonas Nordstr�m 2005-04-18 14:34:58 UTC
The podcast http://dagensskiva.com/rss/chronicles-rss.asp doesn't work with the
plugin "Podcast Browser".

Error message:
Failed to parse <http://dagensskiva.com/rss/chronicles-rss.asp>
unknown encoding at line 1, column 30, byte 30 at /PerlApp/XML/Parser.pm line 187

This xml file uses the encoding according to:
<?xml version="1.0" encoding="windows-1252" ?>
Comment 1 Dave Cohen 2005-04-18 21:57:37 UTC
I can reproduce this bug, and I've done some research but have not figured out
how to fix it.

We ship with the required encoding file, in the CPAN/XML/Parser/Encodings
directory.  And the variable @XML::Parser::Expat::Encoding_Path includes that
directory.  The encoding should be found but seems not to be.

I've used the debugger to step into XML::Parser::Expat.pm line 475, where
ParseString() is called.  But this method cannot be stepped into, and it
produces the error.

Comment 2 Jonas Nordstr�m 2005-04-20 23:36:14 UTC
I made dagensskiva.com change their encoding to iso-8859-1, so the rss at
http://dagensskiva.com/rss/chronicles-rss.asp now works. The problem with
encoding  windows-1252 in slimserver still remains though.
Comment 3 Dave Cohen 2005-04-21 18:47:00 UTC
Created attachment 440 [details]
rss feed with windows encoding

Now if we could just get everyone to stop using Windows (encodings)...

I'm attaching a copy of the feed with the windows encoding, as it could come in
handy for reproducing the bug.
Comment 4 Vidur Apparao 2005-06-30 14:53:36 UTC
Dave, any chance that you could continue looking at this for 6.1?
Comment 5 Dan Sully 2005-07-19 14:16:25 UTC
We're looking into a few options here - but need to push this off post 6.1
Comment 6 Blackketter Dean 2005-08-31 14:01:01 UTC
Dave's probably not going to get to this.
Comment 7 Dan Sully 2005-09-12 12:38:12 UTC
I've tested with a bunch of Windows-1252 encoded feeds, and we appear to be doing the correct thing 
now.
Comment 8 Blackketter Dean 2005-09-19 11:45:55 UTC
Still seeing this here:  http://www.pod-porn.com/xml/rss.xml
Comment 9 Dan Sully 2005-09-19 11:57:41 UTC
I am unable to reproduce this with either the Podcast plugin or RSS ticker.

What version of perl is the customer using?

What version of SlimServer?
Comment 10 Blackketter Dean 2005-09-19 12:06:23 UTC
I just reproduced it in the trunk.
Comment 11 Blackketter Dean 2005-09-19 12:22:15 UTC
Here's my error:

2005-09-19 11:43:41.8244 RssNews failed to parse feed <http://www.pod-porn.com/xml/rss.xml> 
because:

unknown encoding at line 1, column 30, byte 30 at /Users/dean/slim/server/CPAN/XML/Parser.pm line 
187
2005-09-19 11:43:50.8565 Podcast: async request: http://www.pod-porn.com/xml/rss.xml
2005-09-19 11:43:51.3561 Podcast: got http://www.pod-porn.com/xml/rss.xml
2005-09-19 11:43:51.3576 Podcast: content type is text/xml
2005-09-19 11:43:51.3626 Podcast: failed to parse feed because:

unknown encoding at line 1, column 30, byte 30 at /Users/dean/slim/server/CPAN/XML/Parser.pm line 
187
Comment 12 Dan Sully 2005-09-19 13:25:30 UTC
I've done more work to make this not crash as of subversion change 4362

It will be in the 2005-09-20 nightly.