Skip to content

feat(skill): add path traversal review#2599

Closed
452740336 wants to merge 1 commit into
UnitOneAI:mainfrom
452740336:codex/path-traversal-review-skill
Closed

feat(skill): add path traversal review#2599
452740336 wants to merge 1 commit into
UnitOneAI:mainfrom
452740336:codex/path-traversal-review-skill

Conversation

@452740336

Copy link
Copy Markdown

What This PR Does

Adds a new path-traversal-review AppSec skill for issue #286.

The skill covers:

  • JavaScript/TypeScript and Python path traversal review flows
  • file-serving, file-read/write, object-key, and archive-extraction sinks
  • separator-safe containment checks with path.relative and Path.relative_to
  • vulnerable and benign fixtures for direct file reads and archive extraction
  • a lightweight verification script for the skill assets
  • index.yaml registration and role mappings

This contribution was prepared with Codex assistance and reviewed locally against the repository's validation expectations.

Framework References

  • CWE-22: Improper Limitation of a Pathname to a Restricted Directory
  • CWE-23: Relative Path Traversal
  • CWE-36: Absolute Path Traversal
  • OWASP path traversal guidance
  • Node.js path.resolve, path.relative, and path.isAbsolute
  • Python pathlib.Path.resolve and Path.relative_to

Testing

Validated locally with:

@452740336 452740336 requested a review from kamalsrini as a code owner June 15, 2026 03:02
@github-actions github-actions Bot added the needs-approved-issue PR has no linked maintainer-approved issue label Jun 15, 2026
@github-actions

Copy link
Copy Markdown

Thanks for the submission! 🙏 SecuritySkills is now issue-first: contributions need a linked issue that a maintainer has marked approved before a PR is opened.

Please open an issue describing the skill, wait for the approved label, then reopen this PR with Closes #<issue> in the description. The PR template lists everything we'll look for (including an independently runnable reproduction).

@github-actions github-actions Bot closed this Jun 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-approved-issue PR has no linked maintainer-approved issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant