security_monkey
security_monkey copied to clipboard
Hardcoded reference to Riot Games
https://github.com/Netflix/security_monkey/blob/4d198ad29b2f7a828ce130d222c6e76e71b9ce11/scripts/secmonkey_auto_install.sh#L606
Seeing as this is a netflix product, the only hardcoding I would expect would be for Netflix.
Either way. This should probably be configurable
That should be configurable. Please feel free to submit a PR to fix it.
That should be configurable. Please feel free to submit a PR to fix it.
@mikegrima Can I create a PR for this? I think we can make it configurable by passing one extra argument like -o
to mention the company name similarly what is being done for $website
using positional parameter -w|--website
. If the value is not set we can set it to some default value during the creation of ssl certificate. Let me know if my understanding is correct.
@mikegrima here is the code which I am referring to https://github.com/Netflix/security_monkey/blob/4d198ad29b2f7a828ce130d222c6e76e71b9ce11/scripts/secmonkey_auto_install.sh#L167
https://github.com/Netflix/security_monkey/pull/1232 👍🏻