T09 · Insecure Skill Coding Practices
- Location
SKILL.md:165- Finding
Plaintext Configuration Change Logging May Expose Sensitive Values
- Content
View full analysis
- new_value: - reason: - schema_verified: true - doc_source: - user_approved: true - backup: ``` ### Technical Analysis The skill requires every configuration change to be recorded in `~/.openclaw/workspace/logs/config-changes.log`, including the complete previous and new values. It does not require secret detection, redaction, restrictive file permissions, or exclusion of sensitive configuration paths. If a modified configuration value contains an API key, authentication token, password, private endpoint credential, or another secret, that value will be duplicated into a persistent plaintext log. Recording both `old_value` and `new_value` can expose current credentials as well as credentials that were rotated because they were suspected of compromise. This expands the sensitive-data footprint beyond `openclaw.json`. Security controls applied to the primary configuration file may not apply to a log under the workspace directory, which may be included in backups, support bundles, repository commits, synchronization systems, or broad workspace access. ### Attack Path 1. A user requests a legitimate change to a secret-bearing configuration value. 2. The skill follows the mandatory logging procedure after applying the change. 3. The complete previous and new secret values are appended to `~/.openclaw/workspace/logs/config-changes.log`. 4. A local user, process, plugin, backup service, or synchronization tool with access to the workspace log reads ...[truncated 877 chars]- Remediation
View remediation
