Skip to content

Commit 501efb9

Browse files
Issue-1398: set umask when launching browsers (#1399)
* Issue-1398: set umask when launching browsers * Issue-1398: set umask when launching browsers * Issue-1398: comment reasoning and function of calling umask * Issue-1398: set umask when launching browsers * Issue-1398: comment reasoning and function of calling umask Co-authored-by: Diego Molina <[email protected]>
1 parent db72541 commit 501efb9

File tree

2 files changed

+7
-0
lines changed

2 files changed

+7
-0
lines changed

NodeChrome/wrap_chrome_binary

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,10 @@ mv "$WRAPPER_PATH" "$BASE_PATH"
66

77
cat > "$WRAPPER_PATH" <<_EOF
88
#!/bin/bash
9+
10+
# umask 002 ensures default permissions of files are 664 (rw-rw-r--) and directories are 775 (rwxrwxr-x).
11+
umask 002
12+
913
# Note: exec -a below is a bashism.
1014
exec -a "\$0" "$BASE_PATH" --no-sandbox "\$@"
1115
_EOF

NodeEdge/wrap_edge_binary

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,9 @@ mv "$WRAPPER_PATH" "$BASE_PATH"
77
cat > "$WRAPPER_PATH" <<_EOF
88
#!/bin/bash
99
10+
# umask 002 ensures default permissions of files are 664 (rw-rw-r--) and directories are 775 (rwxrwxr-x).
11+
umask 002
12+
1013
# Note: exec -a below is a bashism.
1114
exec -a "\$0" "$BASE_PATH" --no-sandbox "\$@"
1215
_EOF

0 commit comments

Comments
 (0)