Commit4ba6ca3

authored

fix: always open app as logged-in user in postinstall script (#216)

This PR prevents issues like:<img width="428" height="171" alt="Screenshot 2025-07-30 at 3 57 07 pm" src="https://github.com/user-attachments/assets/729055b8-bc9b-43fc-9e95-ed9faa384872" />which occur when the app is launched as root. This can happen when the installer scripts are run as root, which is the case when deploying Coder Desktop over MDM. (As a convenience, we re-open the app if it was open before the installer was ran.)Of note is that on macOS, it is not sufficient to just run open with `sudo -u`, as that does not use the execution context of the user. Seehttps://developer.apple.com/forums/thread/78332Reports of the bug in other programs:(with an incorrect solution)https://community.zoom.com/t5/Zoom-Meetings/A-keychain-cannot-be-found-to-stoer-quot-Zoom-quot/m-p/51059 https://displaylink.org/forum/showthread.php?p=97176

1 parent8d9f844 commit4ba6ca3Copy full SHA for 4ba6ca3

File tree

-1

lines changed

-1

lines changed

Lines changed: 4 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -32,7 +32,10 @@ spctl -avvv "/Applications/Coder Desktop.app/Contents/Library/SystemExtensions/c`
`32`	`32`	`# Restart Coder Desktop if it was running before`
`33`	`33`	`if [-f"$RUNNING_MARKER_FILE" ];then`
`34`	`34`	`echo"Starting Coder Desktop..."`
`35`		`- open -a"Coder Desktop"`
	`35`	`+# When deploying the app via MDM, this script runs as root. The app cannot`
	`36`	`+# function properly when launched as root.`
	`37`	`+ currentUser=$(/usr/bin/stat -f"%Su" /dev/console)`
	`38`	`+ /bin/launchctl asuser"$( /usr/bin/id -u"$currentUser")" /usr/bin/open"/Applications/Coder Desktop.app"`
`36`	`39`	`rm"$RUNNING_MARKER_FILE"`
`37`	`40`	`echo"Coder Desktop started."`
`38`	`41`	`fi`

Comments

(0)