agiresearch / OpenAGI

OpenAGI: When LLM Meets Domain Experts
http://aios.foundation
MIT License
1.93k stars 161 forks source link

Exposed API Key #23

Closed nahatx closed 1 year ago

nahatx commented 1 year ago

https://github.com/agiresearch/OpenAGI/blob/c1ce7595c8e1cfea1bbb9d83beca7b0bb5893360/open_tasks/tools/customized_tools.py#L26

Howdy,

Do not expose your API key in your source code.

Instead, consider storing it an environment variable, or using a key management service, or even software such as Vault. Then be sure to rotate your API key after it is safely stored, or else it may still be recoverable in your commit history.

Also, see Openai's best practices page for api keys for more details: https://help.openai.com/en/articles/5112595-best-practices-for-api-key-safety

TobyGE commented 1 year ago

Thank you for your helpful suggestions. I have disabled the API key prior to the release. Nonetheless, I appreciate your support and concern.