T09 · Insecure Skill Coding Practices
- Location
scripts/geocode.sh:22- Finding
Unrestricted geocoding endpoint override enables server-side request forgery
- Content
View full analysis
- Remediation
View remediation
Security audit
Security checks for vulnerabilities and agentic risk
This reverse-geocoding skill is mostly coherent, but it needs review because its script can be redirected to arbitrary network endpoints through an environment setting.
Review before installing in shared, automated, or network-sensitive environments. Use it only if you are comfortable sending queried coordinates to geocode.com.cn and can keep GEOCODE_BASE_URL fixed to a trusted HTTPS endpoint; avoid letting untrusted prompts, users, or configuration set that variable.
scripts/geocode.sh:22Unrestricted geocoding endpoint override enables server-side request forgery
The skill invokes shell-based functionality via curl but does not declare any explicit tool scope such as permissions or allowed-tools. That creates an avoidable policy gap: an agent may execute network-capable shell commands without clear restriction or review boundaries, increasing the chance of unintended command use or abuse in downstream implementations.
Data is being sent to an external URL. This could be legitimate telemetry or data exfiltration. Manual review is recommended.
## Public API Limits
- Use public endpoints only for low-frequency, interactive lookups.
- Send an identifying `User-Agent`; do not use default curl UA for repeated calls.
- Do not loop, bulk geocode, or aggressively retry against the public endpoint.
## Commands
The script defaults to https://geocode.com.cn and its user-facing messages describe that provider as the expected service, which imposes a specific regional service choice by default. This is a natural-language locale/policy concern because users are not offered an explicit opt-in or rationale for the China-specific provider selection.