T09 · Insecure Skill Coding Practices
- Location
scripts/publish-check.sh:82- Finding
Sensitive credentials are reproduced in terminal and chat scan reports
- Content
View full analysis
/dev/null || true) ``` `scripts/publish-check.sh:89-97`: ```bash if echo "$file_base" | grep -qi '\.env'; then while IFS= read -r line; do [ -z "$line" ] && continue [[ "$line" =~ ^[[:space:]]*# ]] && continue [[ "$line" =~ ^[[:space:]]*$ ]] && continue is_placeholder "$line" && continue log_finding "CRITICAL" ".env Actual Value" "$file_base" "N" ".env contains actual value: $(echo "$line" | head -c 80)" "Delete .env or replace all values with placeholders" done < "$file" fi ``` `scripts/publish-check.sh:104-112`: ```bash while IFS= read -r match; do [ -z "$match" ] && continue local line_num="${match%%:*}" local content="${match#*:}" is_placeholder "$content" && continue log_finding "CRITICAL" "Hardcoded Bearer Token" "$file_base" "$line_num" "Hardcoded Bearer token: $(echo "$content" | head -c 80)" "Use environment variable or placeholder" done < <(grep -nE 'Authorization:[[:space:]]*Bearer[[:space:]]+[A-Za-z0-9_./+-]{8,}' "$file" 2>/dev/null || true) ``` `SKILL.md:65-67`: ```markdown ### Step 3: Output the Full Report **Output the complete scan report to the chat.** Do not summarize or omit anything. ``` ### Technical Analysis The scanner places matching password ...[truncated 2900 chars]- Remediation
View remediation
