-
Notifications
You must be signed in to change notification settings - Fork 84
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
Issue: Jamf Pro Policy Install - Passed Parameter API Creds Plain Text Exposed in Policy #37
Comments
I added a piece to the script to use the encrypted strings to pass an encrypted version of the password. It might be useful if we could get this functional permanently added to the script to be more secure. |
It's my understanding that the create and read permissions are needed to acquire the computer's jamf pro ID. Couldn't we avoid needing to reach out to the jamf pro API to get that information by piping the jamf recon binary into awk to grab the ID that gets printed at the end? something like: jamf recon | awk -F ">" '{print $2}' | awk -F "<" '{print $1}' should return the computers jamf pro ID and avoid the need to do an API call for that info. edit: this doesn't really fix the problem with storing the API creds in plain text, but it should reduce security risk by eliminating the need for unnecessary API permissions. |
@blakeusblade @iDrewbs I am not a fan of encrypted strings, as it adds complexity without providing any real security value. You either have a secret visible in Jamf Pro (where arguably only your trusted administrators should have access) or you put different clear text secrets (hash/salt) hard-coded in to the @rchase254 I am aware this is likely a better method, but at this point for |
Using Jamf Pro here.
Passing paramaters in thescript to deploy super is great, however exposes the API creds in the policy.
And at present, due to another issue (API permission requires Computers > Create/Read) is somewhat of a security hole.
Issue: API - Computers Create and Read Removed - Super doesnt run
#36
The text was updated successfully, but these errors were encountered: