[PR #5609] [MERGED] Images: Updated local disk to have open dir perms #6527

New Issue

OVERLORD · 2026-02-05T10:34:45+03:00

OVERLORD commented

2026-02-05 10:34:45 +03:00

📋 Pull Request Information

Original PR: https://github.com/BookStackApp/BookStack/pull/5609
Author: @ssddanbrown
Created: 5/14/2025
Status: ✅ Merged
Merged: 5/17/2025
Merged by: @ssddanbrown

Base: development ← Head: 5605-folder-permissions

📝 Commits (1)

5ab31a8 Images: Updated local disk to have open dir perms

📊 Changes

3 files changed (+33 additions, -1 deletions)

View changed files

📝 app/Config/filesystems.php (+1 -0)
📝 app/Uploads/ImageStorageDisk.php (+2 -1)
➕ tests/Uploads/ImageStorageTest.php (+30 -0)

📄 Description

Sets directories created for images to have open group/all read/execute permissions so that other process users can access the files within, which goes back to how directory permissions were set before PR #5601.

Closes #5605

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/BookStackApp/BookStack/pull/5609 **Author:** [@ssddanbrown](https://github.com/ssddanbrown) **Created:** 5/14/2025 **Status:** ✅ Merged **Merged:** 5/17/2025 **Merged by:** [@ssddanbrown](https://github.com/ssddanbrown) **Base:** `development` ← **Head:** `5605-folder-permissions` --- ### 📝 Commits (1) - [`5ab31a8`](https://github.com/BookStackApp/BookStack/commit/5ab31a81910b580bf3f86377926ef2d1fb58fac6) Images: Updated local disk to have open dir perms ### 📊 Changes **3 files changed** (+33 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `app/Config/filesystems.php` (+1 -0) 📝 `app/Uploads/ImageStorageDisk.php` (+2 -1) ➕ `tests/Uploads/ImageStorageTest.php` (+30 -0) </details> ### 📄 Description Sets directories created for images to have open group/all read/execute permissions so that other process users can access the files within, which goes back to how directory permissions were set before PR #5601. Closes #5605 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

OVERLORD added the pull-request label 2026-02-05 10:34:45 +03:00

OVERLORD closed this issue

2026-02-05 10:34:49 +03:00

Sign in to join this conversation.

Branches Tags

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/BookStack#6527