T09 · Insecure Skill Coding Practices
- Location
skill.md:80- Finding
SQL Injection Through Unvalidated Workflow Variable in Auto-Numbering Query
- Content
View full analysis
- Remediation
View remediation
Security audit
Security checks for vulnerabilities and agentic risk
This skill is a coherent NocoBase workflow guide, but it gives under-scoped instructions for raw SQL automation and bulk workflow deletion that could modify or remove live business data.
Review this skill carefully before installing. Use it only in environments where the agent is allowed to create and enable NocoBase workflows, and require explicit confirmation before enabling workflows, running raw SQL, or deleting workflows by prefix. Prefer higher-level update nodes or parameterized SQL, test in non-production first, and preview any workflows matched for deletion.
skill.md:80SQL Injection Through Unvalidated Workflow Variable in Auto-Numbering Query
skill.md:149SQL Injection Through Unvalidated Workflow Variable in Scheduled Insurance Update
This section provides a ready-to-run pattern that combines raw SQL data modification with workflow enablement, but does not warn that enabling the workflow can immediately automate future writes to production data. In this skill's context, the danger is elevated because it is operational guidance for automation tooling, so unsafe examples can cause persistent, repeated data changes rather than a one-time mistake.
The skill explicitly recommends bulk cleanup via nb_delete_workflows_by_prefix("AM-") without any warning that this can delete multiple workflows at once. In an agent setting, that omission is dangerous because a user may request cleanup casually and the agent could perform irreversible destructive actions across many workflows.
No suspicious patterns detected.