Stop using process.umask()

Ref nodejs/node#32321
coreyfarrell · Apr 22, 2020 · 52aba53 · 52aba53
1 parent 8e1075d
commit 52aba53
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/mkdirp.js b/mkdirp.js
@@ -13,7 +13,7 @@ function mkdirp(dirpath, customMode, callback) {
     customMode = undefined;
   }
 
-  var mode = customMode || (DEFAULT_DIR_MODE & ~process.umask());
+  var mode = customMode || DEFAULT_DIR_MODE;
   dirpath = path.resolve(dirpath);
 
   fs.mkdir(dirpath, mode, onMkdir);
@@ -46,12 +46,12 @@ function mkdirp(dirpath, customMode, callback) {
         return callback(mkdirErr);
       }
 
-      // TODO: Is it proper to mask like this?
-      if ((stats.mode & MASK_MODE) === mode) {
+      if (!customMode) {
         return callback();
       }
 
-      if (!customMode) {
+      // TODO: Is it proper to mask like this?
+      if ((stats.mode & MASK_MODE) === mode) {
         return callback();
       }