T09 · Insecure Skill Coding Practices
- Location
summarize.py:76- Finding
Unrestricted URL Fetching Enables Server-Side Request Forgery
- Content
View full analysis
- Remediation
View remediation
Security audit
Security checks for vulnerabilities and agentic risk
The skill is a coherent summarizer, but its URL mode can fetch arbitrary addresses without safeguards, which needs review before installation.
Install only if you are comfortable with a summarizer that can read user-specified local files and fetch user-specified URLs. Avoid using it with secret files, private/internal URLs, URLs containing credentials or tokens, or in environments that can reach sensitive internal services unless URL validation or network egress controls are added.
summarize.py:76Unrestricted URL Fetching Enables Server-Side Request Forgery
The skill advertises capabilities that imply local file reading and outbound network access, but it does not declare any explicit tool scope or permission boundaries. This creates a confused-deputy risk where an agent may fetch URLs or read files without clear user consent, increasing the chance of unintended data exposure from local documents or remote requests.
The natural-language description and usage guidance are entirely in Chinese, while the metadata claims 多语言支持(中文/英文). This can be read as forcing a specific language for interaction or documentation without explicit user opt-in or a clear language-selection mechanism.
This code sets a fixed UTF-8 stdout wrapper and presents its docstrings and user-visible status/error text in Chinese, establishing a specific language/locale behavior with no user opt-in or alternate locale support. Under the policy, forcing a specific language without user choice is a natural-language policy violation.
The markdown states that providing a URL will automatically fetch webpage content, but it does not warn users that this triggers an external network request or explain any privacy implications. Users may unknowingly cause requests to third-party sites or send sensitive internal URLs, which can leak browsing intent, tokens in URLs, or internal resource locations.
The manifest description is written entirely in Chinese and does not indicate any user choice of language or locale. This can conflict with language/locale policy requirements when a skill implicitly assumes one language without documenting opt-in or regional scope.
No suspicious patterns detected.