Reject on font loading error

2017-10-23 09:55:05 -06:00
parent c060dedf20
commit 3d48485475
1 changed files with 10 additions and 4 deletions
--- a/src/serve_rendered.js
+++ b/src/serve_rendered.js
@@ -122,12 +122,18 @@ module.exports = function(options, repo, params, id, dataResolver) {
  var existingFonts = {};
  var fontListingPromise = new Promise(function(resolve, reject) {
    fs.readdir(options.paths.fonts, function(err, files) {
+      if (err) {
+        reject(err);
+        return;
+      }
      files.forEach(function(file) {
        fs.stat(path.join(options.paths.fonts, file), function(err, stats) {
-          if (!err) {
-            if (stats.isDirectory()) {
-              existingFonts[path.basename(file)] = true;
-            }
+          if (err) {
+            reject(err);
+            return;
+          }
+          if (stats.isDirectory()) {
+            existingFonts[path.basename(file)] = true;
          }
        });
      });