T09 · Insecure Skill Coding Practices
- Location
SKILL.md:37- Finding
Undocumented Routing Parameter Is Unconditionally Added to API Requests
- Content
View full analysis
Vulnerability Details
File Location:
SKILL.md:37
Vulnerability Type: Request integrity violation and undocumented metadata transmission
Risk Level: MediumVulnerable Code Snippet:
markdown 4. Include `sourceType=5` in every final ComPDF API request. Treat it as an integration routing parameter and include it in request details when the user asks for them.Technical Analysis
The Skill unconditionally requires
sourceType=5to be included in every final ComPDF API request. However, the bundled reference for the supported image-conversion endpoints does not document this numeric value as a request parameter. In those endpoint sections,sourceTypeis instead presented as a response field describing the source file type.This instruction conflicts with the Skill's adjacent requirement to use the exact documented endpoint fields and not infer unsupported options. Because the value is described as an integration routing parameter, it may alter server-side routing, attribution, analytics, billing, or processing behavior without a documented API contract.
The value is fixed rather than attacker-controlled, so this finding does not directly enable arbitrary command execution or privilege escalation. The security concern is undisclosed request modification and the transmission of unnecessary metadata alongside uploaded user files.
Attack Path
- A user invokes the Skill for an image-conversion task.
- The agent selects a documented ComPDF image-conversion endpoint.
- The Skill instruction forces the agent to add
sourceType=5, even though the selected endpoint documentation does not list it as a request field. - The request, uploaded image, API credential, and undocumented routing value are transmitted to ComPDF.
- ComPDF may use the value for routing, attribution, tracking, billing, or other behavior that the user did not explicitly authorize.
Impact Assessment
The issue does n ...[truncated 395 chars]
- Remediation
View remediation
Remediation Suggestions
- Remove the unconditional
sourceType=5requirement unless it is part of the official endpoint contract. - If the parameter is required for a legitimate integration, document its exact server-side meaning, accepted location, data type, and privacy implications.
- Ensure every generated request contains only fields explicitly supported by the selected endpoint documentation.
- Require explicit user disclosure and consent before adding attribution, analytics, affiliate, or routing metadata.
- Add schema validation that rejects undocumented fields before a request is sent.
- Update the bundled API reference with authoritative documentation for the parameter if ComPDF officially supports it.
- Remove the unconditional
