Remove unnecessary process.umask()

Fix #22 Re nodejs/node#32321
isaacs · Apr 3, 2020 · 2d42170 · 2d42170
1 parent 4d59b81
commit 2d42170
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/lib/opts-arg.js b/lib/opts-arg.js
@@ -2,9 +2,9 @@ const { promisify } = require('util')
 const fs = require('fs')
 const optsArg = opts => {
   if (!opts)
-    opts = { mode: 0o777 & (~process.umask()), fs }
+    opts = { mode: 0o777, fs }
   else if (typeof opts === 'object')
-    opts = { mode: 0o777 & (~process.umask()), fs, ...opts }
+    opts = { mode: 0o777, fs, ...opts }
   else if (typeof opts === 'number')
     opts = { mode: opts, fs }
   else if (typeof opts === 'string')

diff --git a/test/opts-arg.js b/test/opts-arg.js
@@ -1,7 +1,6 @@
 const t = require('tap')
 const optsArg = require('../lib/opts-arg.js')
-const umask = process.umask()
-const mode = 0o777 & (~umask)
+const mode = 0o777
 const fs = require('fs')
 
 const defFs = {