~brettgilio/org-webring

1

[PATCH] Support atom feeds that use raw html in the content tag

Jamie Beardslee <jdb@jamzattack.xyz>
Details
Message ID
<87lfh7ixhb.fsf@jamzattack.xyz>
DKIM signature
missing
Download raw message
Usually, atom feeds have:
| <content><![CDATA[ ...]]</content>

However, some feeds just use raw html:
| <content><p>...</p></content>

This is parsed by libxml as normal html, meaning that some feeds would
error, because we were trying to insert a list.

Instead of assuming it's a string, make sure before inserting it.  If
it's a cons, just parse it with dom-strings.
Details
Message ID
<87lfh4gjjq.fsf@debian>
In-Reply-To
<87lfh7ixhb.fsf@jamzattack.xyz> (view parent)
DKIM signature
missing
Download raw message
Jamie Beardslee <jdb@jamzattack.xyz> writes:

> Usually, atom feeds have:
> | <content><![CDATA[ ...]]</content>
>
> However, some feeds just use raw html:
> | <content><p>...</p></content>
>
> This is parsed by libxml as normal html, meaning that some feeds would
> error, because we were trying to insert a list.
>
> Instead of assuming it's a string, make sure before inserting it.  If
> it's a cons, just parse it with dom-strings.

Thank you for the patch, Jamie. It has been merged into master with a
few modifications!

Brett Gilio
Reply to thread Export thread (mbox)