Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Please consider not setting BaseUrl via the postinstall script #1497

Open
flammable opened this issue Feb 22, 2023 · 1 comment
Open

Please consider not setting BaseUrl via the postinstall script #1497

flammable opened this issue Feb 22, 2023 · 1 comment

Comments

@flammable
Copy link

At my last job, we added a second postinstall script, in Munki, to the MunkiReport client installer pkg to do defaults delete /Library/Preferences/MunkiReport.plist BaseUrl because we were setting that via a configuration profile instead.

We had a few rogue users perform a Migration Assistant transfer from their work laptop to a personal one, bringing over MunkiReport and its preferences file. Without those devices checking into Munki properly (to receive an unenroll script), we were powerless to stop them from checking into MunkiReport.

By moving Munki and MunkiReport's preferences to configuration profiles, we were able to perform an enterprise wipe to remove all profiles from the device - thus, even if Munki/MunkiReport were still present on disk, they weren't actively trying to reach our servers anymore.

I know setting the BaseUrl via the built-in postinstall script makes it easier for users to get up and running, but since Munki also requires users to push a profile (or a script running defaults write) to configure a repo URL, I think it's reasonable to expect MunkiReport to do the same.

I think the code for making this happen is here, but it's possible I'm looking in the wrong place. Thanks for considering this!

@tuxudo
Copy link
Contributor

tuxudo commented Feb 22, 2023

Should be supported with the Python 3 version of MunkiReport. When building the client pkg, use -u as an option to not have the BASEURL set by the post install script.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants