From b8a31ef18291402b97079b54bb76f3fc41d146ff Mon Sep 17 00:00:00 2001
From: Frans de Jonge <fransdejonge@gmail.com>
Date: Sat, 15 Feb 2025 10:37:49 +0100
Subject: [PATCH] [plugin] NewsDownloader: don't try to strip <script> tags
 (#13260)

`<script src="etc"></script>` is turned into `<script/>`, which can cause far too much to be stripped.

While that could be dealt with a bit better, for example by first stripping self-closing and then regular, it feels hacky to do so.

See <https://github.com/koreader/koreader/pull/13188#issuecomment-2660362639>.
---
 plugins/newsdownloader.koplugin/epubdownloadbackend.lua | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/plugins/newsdownloader.koplugin/epubdownloadbackend.lua b/plugins/newsdownloader.koplugin/epubdownloadbackend.lua
index 81576eb19..4d0cf5c54 100644
--- a/plugins/newsdownloader.koplugin/epubdownloadbackend.lua
+++ b/plugins/newsdownloader.koplugin/epubdownloadbackend.lua
@@ -333,9 +333,6 @@ function EpubDownloadBackend:createEpub(epub_path, html, url, include_images, me
     local cre = require("libs/libkoreader-cre")
     html = cre.getBalancedHTML(html, 0x0)
 
-    -- Remove all script tags to save a few bytes.
-    html = html:gsub("<script.->.-</script>", "")
-
 --    local sections = html.sections -- Wikipedia provided TOC
     local bookid = "bookid_placeholder" --string.format("wikipedia_%s_%s_%s", lang, phtml.pageid, phtml.revid)
     -- Not sure if this bookid may ever be used by indexing software/calibre, but if it is,