T09 · Insecure Skill Coding Practices
- Location
references/authentication.md:71- Finding
Binance API Credentials Exposed Through Script Storage and Process Arguments
- Content
View full analysis
- Remediation
View remediation
Security audit
Security checks for vulnerabilities and agentic risk
The skill is mostly coherent for Binance P2P lookups, but it handles financial API credentials and includes a shell example that can expose keys if copied with real credentials.
Review this skill before installing if you plan to use personal order history. Use a Binance API key with read-only permission only, enable IP allowlisting where possible, avoid putting real keys into the Bash example or committed .env files, and prefer a secure runtime secret mechanism or in-process client that does not expose secrets in command arguments.
references/authentication.md:71Binance API Credentials Exposed Through Script Storage and Process Arguments
Code accesses credential files (SSH keys, AWS credentials, etc.). This could indicate credential theft attempts.
* Never share your secret key
* Use IP whitelist in Binance API settings
* Enable only required permissions (Enable Reading for P2P order history)
* Store credentials securely in .env file (add to .gitignore)
The skill hardcodes an English locale in generated Binance ad links (/en/adv?...) without checking the user's preferred language or region. This can misroute users to an unexpected localized experience, create confusion, and in some cases weaken trust signals if the displayed page language differs from the user's normal Binance interface.
Skill requests more permissions than appear necessary for its stated functionality. Review if elevated access is justified.
- Secret Key: always mask; show **only last 5**: `***...c123`
### Permission minimization
- Binance API permissions: **Enable Reading** only.
- Do NOT request/encourage trading, withdrawal, or modification permissions.
### Storage guidance