Skip to content

Thumbnail cannot be displayed when uploading pictures #5605

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
archinfoto opened this issue May 11, 2025 · 2 comments · Fixed by #5609
Closed

Thumbnail cannot be displayed when uploading pictures #5605

archinfoto opened this issue May 11, 2025 · 2 comments · Fixed by #5609

Comments

@archinfoto
Copy link

Describe the Bug

Image
When I upload a picture, the thumbnail does not appear.

Image
I found that the folder permissions are not 755.

Steps to Reproduce

See Describle

Expected Behaviour

The permissions were not set correctly when generating the image folder.

Screenshots or Additional Context

No response

Browser Details

google chrome

Exact BookStack Version

V25.02.4

@ssddanbrown
Copy link
Member

Thanks for raising @archinfoto,
Looks to be due to the changes made in #5601 which were part of BookStack v25.02.3.
Probably won't affect many instances where php & the web-server run as the same user, but will be one to patch soon as there's also going to be many instances with a multi-user setup like the one here. Will create a new patch release milestone and assign this to it.

Think we'll need to perform the folder creation manually so we have control over the directory permissions (as we have done for file creation).

@tobast
Copy link

tobast commented May 16, 2025

Hello,

I am indeed affected by this bug on a fresh install at v25.02.4. Thank you for your quick handling of this issue!

Just as a side note, this probably affects many deployments using php-fpm with a pool dedicated to the application, which is usually considered a good practice.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging a pull request may close this issue.

3 participants