Planet returns an XML error for a feed that core aggregator has no problem with

eean - December 23, 2007 - 19:08
Project:Planet
Version:6.x-1.3
Component:Code
Category:bug report
Priority:critical
Assigned:swe3tdave
Status:needs review
Description

Planet returns the error "Failed to parse RSS feed Mark Kretschmann: Invalid document end at line 1."

The feed's url is:
http://amarok.kde.org/blog/feeds/categories/1-markey.rss

The feed is very long since I want to do an initial import of all the old blogs. Perhaps Planet has some limit on how long feeds can be? Its on the same server as Drupal so its certainly not a timeout issue.

The core module Aggregator is able to parse it (thought it only gets the first 4 items...).

#1

eean - December 23, 2007 - 20:12

Limiting the s9y feed to 15 items did allow the import to take place.

#2

eean - December 23, 2007 - 20:22

This feed is having the same problem despite the item limit:
http://amarok.kde.org/blog/feeds/categories/18-freespirit.rss

#3

swe3tdave - March 12, 2009 - 23:09
Version:5.x-1.2» 6.x-1.3
Component:User interface» Code
Priority:normal» critical
Assigned to:Anonymous» swe3tdave

i can confirm that this problem still exist in 6.x-1.3

#4

abraham - March 15, 2009 - 07:47

#5

Nigeria - March 17, 2009 - 00:59

This problem seems to affect all feedburner feeds

#6

swe3tdave - March 17, 2009 - 03:40

Ok, this is an experimental patch, only try this on a test site, i repeat, this is probably highly unstable code... :P

Basically, i switched the xml parsing to the simplepie library. It seems to work, but there are a few bugs that i still have to work out. Unless someone beats me to it.. ;)

AttachmentSize
planet_xml.patch 37.87 KB

#7

swe3tdave - March 17, 2009 - 15:41

This should fix the empty nid in planet_items. The patch is incremental(You apply it after applying planet_xml.patch)

AttachmentSize
planet_xml-2.patch 1.58 KB

#8

swe3tdave - March 19, 2009 - 14:49
AttachmentSize
planet_xml-3.patch 1.15 KB

#9

swe3tdave - March 19, 2009 - 15:56

This should be the last one. Unless someone finds another bug...

AttachmentSize
planet_xml-4.patch 1.66 KB

#10

swe3tdave - March 19, 2009 - 16:07

This patch include all 4. Have Fun.. :)

AttachmentSize
planet_xml.patch 39.58 KB

#11

swe3tdave - March 19, 2009 - 17:48

Small omission, nothing to worry about, but a little bit more helpful... :P

AttachmentSize
planet_xml-readmefix.patch 438 bytes

#12

swe3tdave - March 20, 2009 - 00:57
Status:active» needs review

#13

RockyRoad - June 11, 2009 - 08:51

These problems should be fixed in planet-6.x-1.x-dev release.
I leave the issue open until tests confirm it.

 
 

Drupal is a registered trademark of Dries Buytaert.