T09 · Insecure Skill Coding Practices
- Location
SKILL.md:25- Finding
Cleartext HTTP exposes submission data and report update tokens
- Content
View full analysis
Vulnerability Details
File Location:
SKILL.md, lines 25–35
Vulnerability Type: Cleartext transmission of sensitive report data and bearer-style tokens
Risk Level: Mediumbash curl -s -X POST reviewtimes.fyi/api/v1/reports -H 'content-type: application/json' \ -d '{"store":"claude-plugins","submitted_on":"2026-09-01","status":"waiting"}'Save the returned
token(for example in the project's notes) and update it when the store answers:bash curl -s -X PATCH reviewtimes.fyi/api/v1/reports/$TOKEN -H 'content-type: application/json' \ -d '{"status":"approved","decided_on":"2026-09-20"}'Technical Analysis
The documented
curlcommands use scheme-less URLs. In this context,curltreats the endpoint as HTTP rather than explicitly requiring authenticated HTTPS transport. The POST request transmits the user's store, submission date, and review status. The PATCH request places the server-issued update token in the URL and sends updated report information over the same unencrypted transport.This issue is reachable when a user authorizes reporting a real submission and the agent follows the documented commands. An on-path attacker positioned between the user's environment and
reviewtimes.fyicould inspect or modify HTTP traffic. The instructions do require the user's go-ahead before creating a report, but that confirmation does not protect the resulting network traffic.Attack Path
- The user authorizes the Skill to report a real store submission.
- The agent executes the documented POST command using the scheme-less endpoint.
- The request travels over cleartext HTTP, allowing an on-path attacker to observe the submission metadata or alter the request and response.
- The remote response returns a token used to update the report.
- When the agent later executes the documented PATCH command, the token appears in the cleartext request URL.
- The attacker captures the t ...[truncated 596 chars]
- Remediation
View remediation
Remediation Suggestions
- Replace every scheme-less endpoint with an explicit HTTPS URL:
https://reviewtimes.fyi/api/v1/reportshttps://reviewtimes.fyi/api/v1/reports/$TOKEN
- Require TLS certificate verification and do not add options such as
--insecure. - Avoid storing the returned token in ordinary project notes, which may be committed, shared, or exposed to unrelated project tooling.
- Store the token in an OS credential store or a permission-restricted file outside the project tree.
- Prefer sending update credentials in an authorization header rather than embedding them in the URL, if the API supports that mechanism.
- Document token revocation or report-token rotation procedures in case a token is exposed.
- Replace every scheme-less endpoint with an explicit HTTPS URL:
