Skip to content

fix(security): validate job IDs before filesystem use - #115

Merged
McanKul merged 1 commit into
developmentfrom
fix/validate-job-ids
Oct 4, 2026
Merged

McanKul merged 1 commit into
developmentfrom
fix/validate-job-ids

Conversation

@McanKul

@McanKul McanKul commented Oct 4, 2026

Copy link
Copy Markdown
Owner

Summary

  • reject malformed job IDs before they reach temporary work paths
  • apply the check consistently to PDF and render jobs
  • cover traversal, separators, empty values, and length limits

Validation

  • cargo test --manifest-path src-tauri/Cargo.toml --lib commands::jobs::tests::job_ids_cannot_leave_the_work_folder
  • cargo check --manifest-path src-tauri/Cargo.toml

Small reviewed groundwork for the M4 text-editing work. Targets development only.

@McanKul
McanKul merged commit a52da45 into development Oct 4, 2026
2 checks passed
@McanKul
McanKul deleted the fix/validate-job-ids branch October 4, 2026 10:50
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.

1 participant