This problem is actually a simplepie parser problem and has already been documented by csevb10 on the simplepie bug report/form, but i thought it would also be good to list the problem on the simplefeed issue tracker as well for those who may not bother searching the simplepie bug reports.

I just encountered these two problem today when trying to add my first feed to the latest simplefeed dev. One was an error in simplepie on fclose(), the other a problem with simplepie not correctly parsing the header and body of a feed (the body of the feed gets detected as part of the header) so simplepie doesn't see the feed as a proper feed.

This is a problem with SimplePie parser version "Razzleberry++"... not sure if they have any actual version numbers for this.. but the version name is in the simplepie.inc file. A patch is attached, but to fix it manually (two line fix) refer to: http://simplepie.org/support/viewtopic.php?id=853

CommentFileSizeAuthor
simplepie.patch585 bytesdayre

Comments

csevb10’s picture

csevb10 commented
Assigned: Unassigned » csevb10
Status: Needs review » Fixed

:-) Yep. Sorry, I should have just fixed this in our simplepie.inc so that no one had an issue with it. I was kinda hoping for a quick turnaround from simplepie, but it hasn't gone in yet. I made the quick corrections so that it at least works for fsockopen style connections. This shouldn't be a problem anymore.

m3avrck’s picture

m3avrck commented
Status: Fixed » Reviewed & tested by the community

This change has been reverted.

Using the latest simplepie now and working with those devs to get this change into their core ASAP.

Please use this patch if you are encountering problems as we fix the SimplePie library at the core. Thanks!

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented

Incidentally, this patch also seems to solve the problem of an erroneous "Could not reach feed" error on servers using fsockopen. Thanks for the fix!

m3avrck’s picture

m3avrck commented

Waiting on SimplePie to commit:

http://simplepie.org/support/viewtopic.php?pid=3001#p3001

They will be fixing very soon they said, as there might be some other issues with the patch they need to address at the same time.

m3avrck’s picture

m3avrck commented
Status: Reviewed & tested by the community » Fixed

This has been fixed in r658 of SimplePie and has now been committed.

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)