fix: don't pipe stderr to stdout (#206)

microlinkhq · Sep 15, 2024 · 6b6b4cb · 6b6b4cb
1 parent 6425173
commit 6b6b4cb
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/index.js b/src/index.js
@@ -57,7 +57,7 @@ const fetch = PCancelable.fn(
         if (mutoolPath && contentType === 'application/pdf') {
           const file = getTemporalFile(url, 'pdf')
           await writeFile(file.path, res.body)
-          return (await $(`mutool draw -q -F html ${file.path}`)).stdout
+          return (await $(`${mutoolPath} draw -q -F html ${file.path}`)).stdout
         }
 
         return contentType.startsWith('text/html') || !isMediaUrl(url)
@@ -191,7 +191,7 @@ const defaultGetTemporalFile = (url, ext) => {
 const defaultMutoolPath = () =>
   (() => {
     try {
-      return execSync('which mutool').toString().trim()
+      return execSync('which mutool', { stdio: 'pipe' }).toString().trim()
     } catch (_) {}
   })()