fix: add sudo support for listFiles and improve permission error handling #512

Merged

ZacharyZcR merged 3 commits from feat/file-manager-sudo-support into dev-1.10.1 2026-01-15 10:54:27 +11:00

Conversation 0 Commits 3 Files Changed 4

+229 -23

ZacharyZcR commented 2026-01-15 08:25:16 +11:00 (Migrated from github.com)

Copy Link

Copy Source

Summary

• Add sudo support for listFiles when accessing permission-denied directories
• Frontend now shows sudo password dialog for directory navigation errors
• Fix permission errors displaying as generic "Failed to load directory" message

Changes

• Backend: Add tryWithSudo() function and permission detection in listFiles
• Frontend: loadDirectory returns success/failure status, proper needsSudo handling
• Add translation keys for permission-related messages

## Summary - Add sudo support for `listFiles` when accessing permission-denied directories - Frontend now shows sudo password dialog for directory navigation errors - Fix permission errors displaying as generic "Failed to load directory" message ## Changes - Backend: Add `tryWithSudo()` function and permission detection in `listFiles` - Frontend: `loadDirectory` returns success/failure status, proper `needsSudo` handling - Add translation keys for permission-related messages

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

bug

Something isn't working

dependencies

Pull requests that update a dependency file

docker

Pull requests that update docker code

documentation

Improvements or additions to documentation

duplicate

This issue or pull request already exists

enhancement

New feature or request

github_actions

Pull requests that update GitHub Actions code

good first issue

Good for newcomers

help wanted

Extra attention is needed

invalid

This doesn't seem right

javascript

Pull requests that update javascript code

question

Further information is requested

security

Questions about network security

wontfix

This will not be worked on

No labels

Milestone

No items

No Milestone

Projects

Clear projects

No projects

Assignees

Clear assignees

dennii

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: dennii/Termix#512