Skip to content

Security: constrain local file access paths#8

Merged
BlueVenn6 merged 5 commits into
mainfrom
fix/codeql-127-alerts
Jul 21, 2026
Merged

Security: constrain local file access paths#8
BlueVenn6 merged 5 commits into
mainfrom
fix/codeql-127-alerts

Conversation

@BlueVenn6

Copy link
Copy Markdown
Owner

Summary

  • constrain project-internal paths to approved storage roots, including legacy project asset roots
  • require native desktop path approval for create/open/migrate/backup/restore filesystem operations
  • use server-generated project, restore, image, and reference asset filenames
  • avoid logging security scanner labels or credential-scan result fields that CodeQL treats as sensitive values
  • add traversal, absolute-path escape, asset lookup, and desktop path authorization regression tests

Verification

  • Python compileall: pass
  • Python unittest: 62/62 pass
  • frontend ESLint: pass
  • Vite production build: pass
  • npm audit: 0 vulnerabilities
  • repository hygiene check: pass

The DPAPI settings-storage alert is expected to remain until separately reviewed: the value written to JSON is Windows CryptProtectData ciphertext, not plaintext.

@BlueVenn6
BlueVenn6 merged commit bb191af into main Jul 21, 2026
5 checks passed
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