fixes an issue where cleanup trigger was potentially being run in a c…

…ontext that didn't have permissions to clean up the files.
macadmins · Mar 27, 2024 · f953005 · f953005
1 parent 71bd8fe
commit f953005
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 7 deletions.
diff --git a/Outset/Outset.swift b/Outset/Outset.swift
@@ -207,7 +207,7 @@ struct Outset: ParsableCommand {
                     processItems(loginEveryDir)
                 }
                 if !folderContents(path: loginOncePrivilegedDir).isEmpty || !folderContents(path: loginEveryPrivilegedDir).isEmpty {
-                    FileManager.default.createFile(atPath: loginPrivilegedTrigger, contents: nil)
+                    createTrigger(loginPrivilegedTrigger)
                 }
             }
 
@@ -237,18 +237,13 @@ struct Outset: ParsableCommand {
                     let currentUser = NSUserName()
                     if consoleUser == currentUser {
                         processItems(onDemandDir)
+                        createTrigger(cleanupTrigger)
                     } else {
                         writeLog("User \(currentUser) is not the current console user. Skipping on-demand run.")
                     }
                 } else {
                     writeLog("No current user session. Skipping on-demand run.")
                 }
-                FileManager.default.createFile(atPath: cleanupTrigger, contents: nil)
-                DispatchQueue.main.asyncAfter(deadline: .now() + 0.5) {
-                    if checkFileExists(path: cleanupTrigger) {
-                        pathCleanup(pathname: cleanupTrigger)
-                    }
-                }
             }
         }
 

diff --git a/Outset/Utils/FileUtils.swift b/Outset/Utils/FileUtils.swift
@@ -169,6 +169,10 @@ func deletePath(_ path: String) {
     }
 }
 
+func createTrigger(_ path: String) {
+    FileManager.default.createFile(atPath: path, contents: nil)
+}
+
 func mountDmg(dmg: String) -> String {
     // Attaches dmg and returns the path
     let cmd = "/usr/bin/hdiutil attach -nobrowse -noverify -noautoopen \(dmg)"