Wrap post content in CDATA — sourcehut lists

Umar Getagazov: 1
 Wrap post content in CDATA

 1 files changed, 1 insertions(+), 1 deletions(-)

Export patchset (mbox)

How do I use this?

Copy & paste the following snippet into your terminal to import this patchset into git:

curl -s https://lists.sr.ht/~bitfehler/public-inbox/patches/33925/mbox | git am -3

Learn more about email & git

Reply to thread View this thread in the archives

[PATCH blog] Wrap post content in CDATA Export this patch

Umar Getagazov <umar@handlerug.me>

1 year, 4 months ago

---
I also noticed the RSS feed on the live website is missing post content
altogether. Is that intentional?

Conrad Hoffmann <ch@bitfehler.net>

1 year, 4 months ago

Not at all. I noticed it went missing at some point, but didn't have 
time to investigate. Turns out the mkrss script was still using the 
stock markdown renderer, which is no longer installed on the build 
image. Fixed it to use cmos like for the actual pages, the content is 
back now. Thanks for nudging me on that, too :)

Cheers,
Conrad

 bin/mkrss | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/bin/mkrss b/bin/mkrss
index aeb15f6..9c9e9d7 100755
--- a/bin/mkrss
@@ -35,7 +35,7 @@ for post in $POSTS; do
       <link>https://bitfehler.srht.site/${link}</link>
       <pubDate>$(date -d "${date}" -R)</pubDate>
       <guid>https://bitfehler.srht.site/${link}</guid>
-      <description>$(render_without_title "${post}" | LANG=C sed "s/[\x80-\xFF]//g" | html_encode)</description>
+      <description><![CDATA[$(render_without_title "${post}")]]></description>
     </item>
 EOF
 done
-- 
2.32.1 (Apple Git-133)

Conrad Hoffmann <ch@bitfehler.net>

1 year, 4 months ago
Hey there!

Sorry for the late response, I was out on vacation. Thanks for your 
patch, I applied it. Also:

~bitfehler/public-inbox

blog: Wrap post content in CDATA v1 PROPOSED

[PATCH blog] Wrap post content in CDATA Export this patch