Skip to content

fix: address security issues in actor development skill#21

Merged
vystrcild merged 1 commit intomainfrom
fix-security-issues
Feb 24, 2026
Merged

fix: address security issues in actor development skill#21
vystrcild merged 1 commit intomainfrom
fix-security-issues

Conversation

@patrikbraborec
Copy link
Copy Markdown
Collaborator

Summary

  • Remote Code Execution (CRITICAL): Removed curl | bash and irm | iex install patterns. CLI installation now only recommends npm install -g apify-cli and brew install apify-cli.
  • Credential Safety (HIGH): Replaced apify login -t $APIFY_TOKEN with environment-variable-based auth and interactive login. Added warnings against passing tokens as CLI arguments.
  • Indirect Prompt Injection (HIGH): Added a dedicated Security section with rules for sanitizing crawled data, isolating credentials from data pipelines, and never executing scraped content.
  • External Downloads (MEDIUM): Added guidance to verify package names before installing and noted lockfile usage for reproducible installs.

🤖 Generated with Claude Code

Remove curl|bash install pattern (RCE risk), replace with package manager
installs. Secure credential handling by removing token CLI args and
preferring env vars/interactive login. Add dedicated Security section for
untrusted web content handling and prompt injection prevention. Add
supply-chain guidance for dependency installation.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@vystrcild vystrcild merged commit d28d583 into main Feb 24, 2026
2 checks passed
@vystrcild vystrcild deleted the fix-security-issues branch February 24, 2026 14:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants