Impact: Developer account security
Experienced: While browsing the code, I noticed in .github/workflows/deploy.yml there were 3 GitHub usernames hard coded into the deploy script.
Expected: These usernames need to be a secret variable. Not only does this make the script more portable, obfusticates the deployment usernames from the general public.
Repro:
- Navigate to https://github.com/burningmantech/ranger-ims-server/blob/master/.github/workflows/deploy.yml
- Observe line 32 and 93
Additional comment:
I guess you could assertain the usernames from the commits; however, hard coding usernames in a script that specifically is designated to deploy is a security risk. It tells prying eyes who to target.