Skip to content

SimplePie: Fix support for feeds with XML preample + DTD#7515

Merged
Alkarex merged 1 commit intoFreshRSS:edgefrom
Alkarex:fix-xml-dtd
Apr 18, 2025
Merged

SimplePie: Fix support for feeds with XML preample + DTD#7515
Alkarex merged 1 commit intoFreshRSS:edgefrom
Alkarex:fix-xml-dtd

Conversation

@Alkarex
Copy link
Member

@Alkarex Alkarex commented Apr 18, 2025

Regression from #4374
fix: #7514
FreshRSS/simplepie#35
Upstream PR: simplepie/simplepie#914

@Alkarex Alkarex merged commit d1f9b6c into FreshRSS:edge Apr 18, 2025
1 check passed
@Alkarex Alkarex deleted the fix-xml-dtd branch April 18, 2025 12:59
Alkarex added a commit to Alkarex/FreshRSS that referenced this pull request Jun 24, 2025
fix FreshRSS#7687

FreshRSS/simplepie#37
Upstream: simplepie/simplepie#915

Partial revert of FreshRSS#7515

HTML entities are normally only allowed in XML when there is a DTD declaring them. SimplePie is even allowing Atom documents with undeclared HTML entities - which I am not sure is on purpose.
Alkarex added a commit that referenced this pull request Jun 24, 2025
fix #7687

FreshRSS/simplepie#37
Upstream: simplepie/simplepie#915

Partial revert of #7515

HTML entities are normally only allowed in XML when there is a DTD declaring them. SimplePie is even allowing Atom documents with undeclared HTML entities - which I am not sure is on purpose.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feed problem] various RSS 0.91 feeds throwing XML_ERR_NAME_REQUIRED

2 participants