Merge pull request #1847 from hackmdio/chore/bump-pandoc-js

Address pandoc related issues
hackmdio · Jan 4, 2024 · 9846143 · 9846143
2 parents cc78023 + cd52cb7
commit 9846143
Show file tree

Hide file tree

Showing 3 changed files with 186 additions and 49 deletions.
diff --git a/lib/note/noteActions.js b/lib/note/noteActions.js
@@ -133,14 +133,17 @@ async function actionPandoc (req, res, note) {
   content = content.replace(/\]\(\//g, '](' + url + '/')
 
   const { exportType } = req.query
+  if (typeof exportType !== 'string') {
+    return res.sendStatus(400)
+  }
+
   const contentType = outputFormats[exportType]
+  if (!contentType) {
+    return res.sendStatus(400)
+  }
 
   try {
     // TODO: timeout rejection
-    if (!contentType) {
-      return res.sendStatus(400)
-    }
-
     await pandoc.convertToFile(content, 'markdown', exportType, path, [
       '--metadata', `title=${title}`, '--sandbox'
     ])