Skip to content

fix(control): fix filesystem permissions for non-root execution of commands#52

Merged
BPR02 merged 12 commits intomainfrom
dev
Mar 20, 2026
Merged

fix(control): fix filesystem permissions for non-root execution of commands#52
BPR02 merged 12 commits intomainfrom
dev

Conversation

@BPR02
Copy link
Copy Markdown
Owner

@BPR02 BPR02 commented Mar 19, 2026

closes #53
This PR fixes certain things related to file permissions. This is important when mounting the bind mounts to a non-user owned folder (e.g. /srv) and using peerstash commands without sudo.

  • peerstash setup fails to generate the restic password
  • peerstash restore cannot restore into the /mnt/peerstash_restore folder
  • peerstash restore cannot delete the /tmp/peerstash/restore folder if it was previously run with sudo
  • peerstash mount cannot mount into /tmp/peerstash_mnt folder
  • peerstash unmount does not complete

Changes:

  • peerstash mount can be used with or without sudo, but will only unmountable with sudo if mounted with root permissions
  • Repos will be unmounted when the container is starting up. This will remove stale repos that were still mounted when the container came down.

@BPR02 BPR02 marked this pull request as draft March 19, 2026 18:24
- it's set up in Dockerfile, but maybe something breaks it
@BPR02 BPR02 changed the title fix(control): allow non-sudo creation of restic password file fix(control): fix filesystem permissions for non-root execution of commands Mar 19, 2026
@BPR02 BPR02 marked this pull request as ready for review March 20, 2026 00:53
@BPR02 BPR02 merged commit 88b6e6b into main Mar 20, 2026
7 checks passed
bpr02-release-bot bot pushed a commit that referenced this pull request Mar 20, 2026
## [0.15.2](peerstash-v0.15.1...peerstash-v0.15.2) (2026-03-20)

### Bug Fixes

* **control:** fix filesystem permissions for non-root execution of commands ([#52](#52)) ([88b6e6b](88b6e6b))
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.

Filesystem Permissions Break Functionality

1 participant