Change chmod from 777 to 755 (#19141)

DmitriiBobreshev · web-flow · commit 42319fabf4e9 · 2023-10-19T13:24:56.000+02:00
-- Changed chmod for the tasks from 777 to 755s
-- Changed Tasks: ContainerBuildV0, ContainerStructureTestV0, DuffleInstallerV0, FuncToolsInstallerV0, OpenPolicyAgentInstallerV0
diff --git a/Tasks/ContainerBuildV0/src/utils.ts b/Tasks/ContainerBuildV0/src/utils.ts
@@ -65,7 +65,7 @@ export async function downloadBuildctl(version: string): Promise<string> {
 
     tl.debug('Buildctl path: ' + buildctlpath);
 
-    fs.chmodSync(buildctlpath, "777");
+    fs.chmodSync(buildctlpath, "755");
     return buildctlpath;
 }
 
diff --git a/Tasks/ContainerBuildV0/task.json b/Tasks/ContainerBuildV0/task.json
@@ -13,7 +13,7 @@
     "author": "Microsoft Corporation",
     "version": {
         "Major": 0,
-        "Minor": 225,
+        "Minor": 230,
         "Patch": 0
     },
     "demands": [],
diff --git a/Tasks/ContainerBuildV0/task.loc.json b/Tasks/ContainerBuildV0/task.loc.json
@@ -13,7 +13,7 @@
   "author": "Microsoft Corporation",
   "version": {
     "Major": 0,
-    "Minor": 225,
+    "Minor": 230,
     "Patch": 0
   },
   "demands": [],
diff --git a/Tasks/ContainerStructureTestV0/task.json b/Tasks/ContainerStructureTestV0/task.json
@@ -13,7 +13,7 @@
     "author": "Microsoft Corporation",
     "version": {
         "Major": 0,
-        "Minor": 219,
+        "Minor": 230,
         "Patch": 0
     },
     "preview": true,
diff --git a/Tasks/ContainerStructureTestV0/task.loc.json b/Tasks/ContainerStructureTestV0/task.loc.json
@@ -13,7 +13,7 @@
   "author": "Microsoft Corporation",
   "version": {
     "Major": 0,
-    "Minor": 219,
+    "Minor": 230,
     "Patch": 0
   },
   "preview": true,
diff --git a/Tasks/ContainerStructureTestV0/testrunner.ts b/Tasks/ContainerStructureTestV0/testrunner.ts
@@ -37,7 +37,7 @@ export class TestRunner {
                     throw new Error(`Download or caching of tool(${runnerPath}) failed`);
                 }
 
-                chmodSync(runnerPath, "777");
+                chmodSync(runnerPath, "755");
                 var start = new Date().getTime();
                 const output: string = this.runContainerStructureTest(runnerPath, this.testFilePath, this.imageName);
                 var end = new Date().getTime();
diff --git a/Tasks/DuffleInstallerV0/src/duffleinstaller.ts b/Tasks/DuffleInstallerV0/src/duffleinstaller.ts
@@ -39,7 +39,7 @@ async function downloadDuffle(version: string): Promise<string> {
     }
 
     const dufflePath = path.join(cachedToolPath, DuffleToolName + getExecutableExtension());
-    fs.chmodSync(dufflePath, '777');
+    fs.chmodSync(dufflePath, '755');
     return dufflePath;
 }
 
diff --git a/Tasks/DuffleInstallerV0/task.json b/Tasks/DuffleInstallerV0/task.json
@@ -12,7 +12,7 @@
   "author": "Microsoft Corporation",
   "version": {
     "Major": 0,
-    "Minor": 228,
+    "Minor": 230,
     "Patch": 0
   },
   "demands": [],
diff --git a/Tasks/DuffleInstallerV0/task.loc.json b/Tasks/DuffleInstallerV0/task.loc.json
@@ -12,7 +12,7 @@
   "author": "Microsoft Corporation",
   "version": {
     "Major": 0,
-    "Minor": 228,
+    "Minor": 230,
     "Patch": 0
   },
   "demands": [],
diff --git a/Tasks/FuncToolsInstallerV0/src/utils.ts b/Tasks/FuncToolsInstallerV0/src/utils.ts
@@ -75,10 +75,10 @@ async function downloadFuncToolsInternal(version: string): Promise<string> {
     }
 
     const funcPath = path.join(cachedToolpath, funcToolName + getExecutableExtension());
-    fs.chmodSync(funcPath, '777');
+    fs.chmodSync(funcPath, '755');
     const gozipPath = path.join(cachedToolpath, 'gozip' + getExecutableExtension());
     if (fs.existsSync(gozipPath)) {
-        fs.chmodSync(gozipPath, '777');
+        fs.chmodSync(gozipPath, '755');
     }
     
     return funcPath;
diff --git a/Tasks/FuncToolsInstallerV0/task.json b/Tasks/FuncToolsInstallerV0/task.json
@@ -13,7 +13,7 @@
   "author": "Microsoft Corporation",
   "version": {
     "Major": 0,
-    "Minor": 224,
+    "Minor": 230,
     "Patch": 0
   },
   "preview": true,
diff --git a/Tasks/FuncToolsInstallerV0/task.loc.json b/Tasks/FuncToolsInstallerV0/task.loc.json
@@ -13,7 +13,7 @@
   "author": "Microsoft Corporation",
   "version": {
     "Major": 0,
-    "Minor": 224,
+    "Minor": 230,
     "Patch": 0
   },
   "preview": true,
diff --git a/Tasks/OpenPolicyAgentInstallerV0/src/utils.ts b/Tasks/OpenPolicyAgentInstallerV0/src/utils.ts
@@ -83,11 +83,11 @@ export async function downloadOpa(version: string): Promise<string> {
     if (!cachedToolpath || !fs.existsSync(opaPath)) {
         const opaPathTmp = path.join(getTempDirectory(), opaToolName + getExecutableExtension());
         taskLib.cp(opaDownloadPath, opaPathTmp, '-f');
-        fs.chmodSync(opaPathTmp, '777');
+        fs.chmodSync(opaPathTmp, '755');
         return opaPathTmp;
     }
 
-    fs.chmodSync(opaPath, '777');
+    fs.chmodSync(opaPath, '755');
     return opaPath;
 }
 
diff --git a/Tasks/OpenPolicyAgentInstallerV0/task.json b/Tasks/OpenPolicyAgentInstallerV0/task.json
@@ -13,7 +13,7 @@
     "author": "Microsoft Corporation",
     "version": {
         "Major": 0,
-        "Minor": 225,
+        "Minor": 230,
         "Patch": 0
     },
     "minimumAgentVersion": "2.144.0",
diff --git a/Tasks/OpenPolicyAgentInstallerV0/task.loc.json b/Tasks/OpenPolicyAgentInstallerV0/task.loc.json
@@ -13,7 +13,7 @@
   "author": "Microsoft Corporation",
   "version": {
     "Major": 0,
-    "Minor": 225,
+    "Minor": 230,
     "Patch": 0
   },
   "minimumAgentVersion": "2.144.0",

Original file line number	Diff line number	Diff line change
`@@ -65,7 +65,7 @@ export async function downloadBuildctl(version: string): Promise<string> {`
`65`	`65`
`66`	`66`	`tl.debug('Buildctl path: ' + buildctlpath);`
`67`	`67`
`68`		`- fs.chmodSync(buildctlpath, "777");`
	`68`	`+ fs.chmodSync(buildctlpath, "755");`
`69`	`69`	`return buildctlpath;`
`70`	`70`	`}`
`71`	`71`
Original file line number	Diff line number	Diff line change
`@@ -37,7 +37,7 @@ export class TestRunner {`
`37`	`37`	throw new Error(`Download or caching of tool(${runnerPath}) failed`);
`38`	`38`	`}`
`39`	`39`
`40`		`- chmodSync(runnerPath, "777");`
	`40`	`+ chmodSync(runnerPath, "755");`
`41`	`41`	`var start = new Date().getTime();`
`42`	`42`	`const output: string = this.runContainerStructureTest(runnerPath, this.testFilePath, this.imageName);`
`43`	`43`	`var end = new Date().getTime();`
Original file line number	Diff line number	Diff line change
`@@ -39,7 +39,7 @@ async function downloadDuffle(version: string): Promise<string> {`
`39`	`39`	`}`
`40`	`40`
`41`	`41`	`const dufflePath = path.join(cachedToolPath, DuffleToolName + getExecutableExtension());`
`42`		`- fs.chmodSync(dufflePath, '777');`
	`42`	`+ fs.chmodSync(dufflePath, '755');`
`43`	`43`	`return dufflePath;`
`44`	`44`	`}`
`45`	`45`
Original file line number	Diff line number	Diff line change
`@@ -75,10 +75,10 @@ async function downloadFuncToolsInternal(version: string): Promise<string> {`
`75`	`75`	`}`
`76`	`76`
`77`	`77`	`const funcPath = path.join(cachedToolpath, funcToolName + getExecutableExtension());`
`78`		`- fs.chmodSync(funcPath, '777');`
	`78`	`+ fs.chmodSync(funcPath, '755');`
`79`	`79`	`const gozipPath = path.join(cachedToolpath, 'gozip' + getExecutableExtension());`
`80`	`80`	`if (fs.existsSync(gozipPath)) {`
`81`		`- fs.chmodSync(gozipPath, '777');`
	`81`	`+ fs.chmodSync(gozipPath, '755');`
`82`	`82`	`}`
`83`	`83`
`84`	`84`	`return funcPath;`