Skip to content

Commit ff2ab22

Browse files
Merge pull request #536 from rohan-stepsecurity/feat/flag/disable-sudo-and-containers
fix: run sudo command only when both disable-sudo and disable-sudo-an…
2 parents 769df4e + b81d650 commit ff2ab22

File tree

1 file changed

+2
-14
lines changed

1 file changed

+2
-14
lines changed

src/cleanup.ts

+2-14
Original file line numberDiff line numberDiff line change
@@ -81,21 +81,9 @@ import { isGithubHosted } from "./tls-inspect";
8181
}
8282

8383
var disable_sudo = process.env.STATE_disableSudo;
84-
if (disable_sudo !== "true") {
85-
try {
86-
var journalLog = cp.execSync("sudo journalctl -u agent.service --lines=1000", {
87-
encoding: "utf8",
88-
maxBuffer: 1024 * 1024 * 10 // 10MB buffer
89-
});
90-
console.log("agent.service log:");
91-
console.log(journalLog);
92-
} catch (error) {
93-
console.log("Warning: Could not fetch service logs:", error.message);
94-
}
95-
}
96-
9784
var disable_sudo_and_containers = process.env.disableSudoAndContainers;
98-
if (disable_sudo_and_containers !== "true") {
85+
86+
if (disable_sudo !== "true" && disable_sudo_and_containers !== "true") {
9987
try {
10088
var journalLog = cp.execSync("sudo journalctl -u agent.service --lines=1000", {
10189
encoding: "utf8",

0 commit comments

Comments
 (0)