Skip to content

Commit 28715bc

Browse files
committed
Fix: make sure all extracted tarballs are r/w enabled
**Summary** Fixes: #992. This issue was supposed to be fixed by #2826 but it was not setting directory permissions wide enough. This patch uses the `readable` and `writable` options provided by the `tar-fs` package which essentially sets everything to `0o777`. **Test plan** N/A
1 parent 4bb34e2 commit 28715bc

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

src/fetchers/tarball-fetcher.js

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -67,8 +67,9 @@ export default class TarballFetcher extends BaseFetcher {
6767
const extractorStream = gunzip();
6868
const untarStream = tarFs.extract(this.dest, {
6969
strip: 1,
70-
dmode: 0o555, // all dirs should be readable
71-
fmode: 0o444, // all files should be readable
70+
// everything should be readable and writeable
71+
readable: true,
72+
writable: true,
7273
chown: false, // don't chown. just leave as it is
7374
});
7475

0 commit comments

Comments
 (0)