fix(filesystem): Don't call toString on binary files

rdkcentral · Jun 6, 2023 · 42746a1 · 42746a1
1 parent cd79adc
commit 42746a1
Showing 1 changed file with 8 additions and 1 deletion.
diff --git a/src/shared/filesystem.mjs b/src/shared/filesystem.mjs
@@ -40,6 +40,8 @@ const readDir = async (ref, options) => {
     return results.sort()
 }
 
+const binFormats = [ '.png', '.jpg', '.gif' ]
+
 const readFiles = (refs, base) => Promise.all(refs.map(ref => readFile(ref)))
                             .then(contents => {
                                 if (!refs || refs.length === 0) {
@@ -63,7 +65,12 @@ const readFiles = (refs, base) => Promise.all(refs.map(ref => readFile(ref)))
 
                                 const results = refs.map(v => [v.substring(index), null])
                                 for (let i=0; i<refs.length; i++) {
-                                    results[i][1] = contents[i].toString()
+                                    if (binFormats.find(suffix => refs[i].endsWith(suffix))) {
+                                        results[i][1] = contents[i]
+                                    }
+                                    else {
+                                        results[i][1] = contents[i].toString()
+                                    }
                                 }
                                 return Promise.resolve(Object.fromEntries(results))
                             })