Switched xml_escape for <![CDATA[]]> for post content (#332)

Merge pull request 332
jekyll · Jan 1, 2022 · 1ac64bc · 1ac64bc
1 parent 385f455
commit 1ac64bc
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/lib/jekyll-feed/feed.xml b/lib/jekyll-feed/feed.xml
@@ -63,7 +63,7 @@
       <id>{{ post.id | absolute_url | xml_escape }}</id>
       {% assign excerpt_only = post.feed.excerpt_only | default: site.feed.excerpt_only %}
       {% unless excerpt_only %}
-        <content type="html" xml:base="{{ post.url | absolute_url | xml_escape }}">{{ post.content | strip | xml_escape }}</content>
+        <content type="html" xml:base="{{ post.url | absolute_url | xml_escape }}"><![CDATA[{{ post.content | strip }}]]></content>
       {% endunless %}
 
       {% assign post_author = post.author | default: post.authors[0] | default: site.author %}
@@ -96,7 +96,7 @@
 
       {% assign post_summary = post.description | default: post.excerpt %}
       {% if post_summary and post_summary != empty %}
-        <summary type="html">{{ post_summary | strip_html | normalize_whitespace | xml_escape }}</summary>
+        <summary type="html"><![CDATA[{{ post_summary | strip_html | normalize_whitespace }}]]></summary>
       {% endif %}
 
       {% assign post_image = post.image.path | default: post.image %}

diff --git a/spec/jekyll-feed_spec.rb b/spec/jekyll-feed_spec.rb
@@ -76,7 +76,7 @@
   end
 
   it "converts markdown posts to HTML" do
-    expect(contents).to match %r!&lt;p&gt;March the second\!&lt;/p&gt;!
+    expect(contents).to match %r!<\!\[CDATA\[<p>March the second\!</p>\]\]!
   end
 
   it "uses last_modified_at where available" do