🔒 Security: Redact potential hardcoded secrets#83
Conversation
There was a problem hiding this comment.
Code Review
This pull request replaces placeholder API keys in JULES_API.md and README.md with 'REDACTED_BY_JULES'. The reviewer recommends reverting these changes, as the original values were clear placeholders for users, and redacting them could lead to confusion.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
|
|
||
| ```bash | ||
| export JULES_API_KEY="your-api-key-here" | ||
| export JULES_API_KEY='REDACTED_BY_JULES' |
There was a problem hiding this comment.
The original value "your-api-key-here" is a placeholder intended to guide users on where to insert their actual API key. Redacting it to 'REDACTED_BY_JULES' makes the documentation less clear and might confuse users into thinking this is a literal value or a system-defined keyword. Since this is not a real secret, it should not be redacted.
| export JULES_API_KEY='REDACTED_BY_JULES' | |
| export JULES_API_KEY="your-api-key-here" |
| Set it as an environment variable — **never commit it**: | ||
| ```bash | ||
| export JULES_API_KEY="your-api-key-here" | ||
| export JULES_API_KEY='REDACTED_BY_JULES' |
There was a problem hiding this comment.
The original value "your-api-key-here" is a placeholder intended to guide users on where to insert their actual API key. Redacting it to 'REDACTED_BY_JULES' makes the documentation less clear and might confuse users into thinking this is a literal value or a system-defined keyword. Since this is not a real secret, it should not be redacted.
| export JULES_API_KEY='REDACTED_BY_JULES' | |
| export JULES_API_KEY="your-api-key-here" |
I found potential secrets in the codebase and redacted them. Please review and rotate these secrets! 🛡️✨