T09 · Insecure Skill Coding Practices
- Location
SKILL.md:82- Finding
State-Changing Configuration Push Misrepresented as a Status Check
- Content
View full analysis
Vulnerability Details
File Location:
SKILL.md, lines 82–86
Vulnerability Type: Unsafe state-changing API operation without explicit authorization
Risk Level: Mediummarkdown 1. Check candidate config status: ``` POST /sse/config/v1/config-versions/candidate:push ``` 2. Monitor push job status:Technical Analysis
The troubleshooting guide describes
POST /sse/config/v1/config-versions/candidate:pushas a method to “Check candidate config status.” However, both the HTTP method and thecandidate:pushaction indicate a state-changing operation that may deploy pending configuration rather than perform a read-only status query.The documented workflow does not require the agent to inspect pending changes, validate the candidate configuration, obtain explicit user approval, use a dry-run mechanism, or establish a rollback plan before initiating the push. This creates a risk that an agent following the skill will mistake a deployment action for a diagnostic query.
Exploitation does not grant privileges beyond those already associated with the configured SCM credentials. Nevertheless, an operator, misleading request, or automated troubleshooting flow could cause the agent to exercise those existing privileges in an unintended and disruptive manner.
Attack Path
- The skill is invoked to diagnose a Prisma Access configuration push failure.
- The agent follows the documented step labeled “Check candidate config status.”
- The agent authenticates to Strata Cloud Manager using the configured
SCM_CLIENT_ID,SCM_CLIENT_SECRET, andSCM_TSG_ID. - The agent sends
POST /sse/config/v1/config-versions/candidate:push. - Pending candidate changes are pushed without a separate, informed authorization step.
- Unreviewed or erroneous changes may alter production connectivity or security enforcement.
Impact Assessment
The operation can act within the configuration scope and ...[truncated 619 chars]
- Remediation
View remediation
Remediation Suggestions
- Replace the push request with a documented read-only endpoint for inspecting candidate configuration or status.
- Clearly label
candidate:pushas a mutating deployment operation rather than a diagnostic status check. - Require explicit user confirmation immediately before any configuration push.
- Display or summarize pending changes and validate them before requesting approval.
- Verify the target tenant, folder, and configuration scope before execution.
- Use a dry-run or validation endpoint where the SCM API supports one.
- Check for concurrent edits and active jobs before initiating a push.
- Monitor the resulting job to completion and report all validation or deployment errors.
- Document a tested rollback or recovery procedure for failed or disruptive pushes.
- Use least-privilege SCM credentials so read-only troubleshooting cannot invoke configuration deployment operations.
