Skip to content

feat: implement secure path validation for downloadManyFiles#2

Open
thiyaguk09 wants to merge 8061 commits intomainfrom
fix/download-directory-path-traversal
Open

feat: implement secure path validation for downloadManyFiles#2
thiyaguk09 wants to merge 8061 commits intomainfrom
fix/download-directory-path-traversal

Conversation

@thiyaguk09
Copy link
Copy Markdown
Owner

  • Adds protection against path traversal (../) using normalized path resolution.
  • Prevents Windows-style drive letter injection while allowing GCS timestamps.
  • Implements directory jail logic to ensure absolute-style paths are relative to destination.
  • Preserves backward compatibility by returning an augmented DownloadResponse array.
  • Automates recursive directory creation for validated nested files.
  • Adds comprehensive 13-scenario test suite for edge-case parity.

Loading
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.