mirror of
https://github.com/BookStackApp/BookStack.git
synced 2026-02-19 11:19:38 +03:00
Unable to upload image #125
Closed
opened 2026-02-04 17:04:15 +03:00 by OVERLORD
·
11 comments
No Branch/Tag Specified
development
release
v25-12
l10n_development
llm_only
vectors
v25-11
docker_env
drawio_rendering
user_permissions
ldap_host_failover
svg_image
prosemirror
captcha_example
fix/video-export
v25.12.6
v25.12.5
v25.12.4
v25.12.3
v25.12.2
v25.12.1
v25.12
v25.11.6
v25.11.5
v25.11.4
v24.11.4
v25.11.3
v25.11.2
v25.11.1
v25.11
v25.07.3
v25.07.2
v25.07.1
v25.07
v25.05.2
v25.05.1
v25.05
v25.02.5
v25.02.4
v25.02.3
v25.02.2
v25.02.1
v25.02
v24.12.1
v24.12
v24.10.3
v24.10.2
v24.10.1
v24.10
v24.05.4
v24.05.3
v24.05.2
v24.05.1
v24.05
v24.02.3
v24.02.2
v24.02.1
v24.02
v23.12.3
v23.12.2
v23.12.1
v23.12
v23.10.4
v23.10.3
v23.10.2
v23.10.1
v23.10
v23.08.3
v23.08.2
v23.08.1
v23.08
v23.06.2
v23.06.1
v23.06
v23.05.2
v23.05.1
v23.05
v23.02.3
v23.02.2
v23.02.1
v23.02
v23.01.1
v23.01
v22.11.1
v22.11
v22.10.2
v22.10.1
v22.10
v22.09.1
v22.09
v22.07.3
v22.07.2
v22.07.1
v22.07
v22.06.2
v22.06.1
v22.06
v22.04.2
v22.04.1
v22.04
v22.03.1
v22.03
v22.02.3
v22.02.2
v22.02.1
v22.02
v21.12.5
v21.12.4
v21.12.3
v21.12.2
v21.12.1
v21.12
v21.11.3
v21.11.2
v21.11.1
v21.11
v21.10.3
v21.10.2
v21.10.1
v21.10
v21.08.6
v21.08.5
v21.08.4
v21.08.3
v21.08.2
v21.08.1
v21.08
v21.05.4
v21.05.3
v21.05.2
v21.05.1
v21.05
v21.04.6
v21.04.5
v21.04.4
v21.04.3
v21.04.2
v21.04.1
v21.04
v0.31.8
v0.31.7
v0.31.6
v0.31.5
v0.31.4
v0.31.3
v0.31.2
v0.31.1
v0.31.0
v0.30.7
v0.30.6
v0.30.5
v0.30.4
v0.30.3
v0.30.2
v0.30.1
v0.30.0
v0.29.3
v0.29.2
v0.29.1
v0.29.0
v0.28.3
v0.28.2
v0.28.1
v0.28.0
v0.27.5
v0.27.4
v0.27.3
v0.27.2
v0.27.1
v0.27
v0.26.4
v0.26.3
v0.26.2
v0.26.1
v0.26.0
v0.25.5
v0.25.4
v0.25.3
v0.25.2
v0.25.1
v0.25.0
v0.24.3
v0.24.2
v0.24.1
v0.24.0
v0.23.2
v0.23.1
v0.23.0
v0.22.0
v0.21.0
v0.20.3
v0.20.2
v0.20.1
v0.20.0
v0.19.0
v0.18.5
v0.18.4
v0.18.3
v0.18.2
v0.18.1
v0.18.0
v0.17.4
v0.17.3
v0.17.2
v0.17.1
v0.17.0
v0.16.3
v0.16.2
v0.16.1
v0.16.0
v0.15.3
v0.15.2
v0.15.1
v0.15.0
v0.14.3
v0.14.2
v0.14.1
v0.14.0
v0.13.1
v0.13.0
v0.12.2
v0.12.1
v0.12.0
v0.11.2
v0.11.1
v0.11.0
v0.10.0
v0.9.3
v0.9.2
v0.9.1
v0.9.0
v0.8.2
v0.8.1
v0.8.0
v0.7.6
v0.7.5
v0.7.4
v0.7.3
0.7.2
v.0.7.1
v0.7.0
v0.6.3
v0.6.2
v0.6.1
v0.6.0
v0.5.0
Labels
Clear labels
🎨 Design
📖 Docs Update
🐛 Bug
🐛 Bug
:cat2:🐈 Possible duplicate
💿 Database
☕ Open to discussion
💻 Front-End
🐕 Support
🚪 Authentication
🌍 Translations
🔌 API Task
🏭 Back-End
⛲ Upstream
🔨 Feature Request
🛠️ Enhancement
🛠️ Enhancement
🛠️ Enhancement
❤️ Happy feedback
🔒 Security
🔍 Pending Validation
💆 UX
📝 WYSIWYG Editor
🌔 Out of scope
🔩 API Request
:octocat: Admin/Meta
🖌️ View Customization
❓ Question
🚀 Priority
🛡️ Blocked
🚚 Export System
♿ A11y
🔧 Maintenance
> Markdown Editor
pull-request
Mirrored from GitHub Pull Request
No Label
🐕 Support
Milestone
No items
No Milestone
Projects
Clear projects
No project
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: starred/BookStack#125
Reference in New Issue
Block a user
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @calverp on GitHub (Jun 30, 2016).
I'll begin by stating that this is very likely user error on my part. I'm running Bookstack from Windows/IIS and everything is working fine except for uploading images. If I drag a file into the upload box, a small thumbnail appears with the name and size of the file, and a red hover over box with a bunch of HTML appears, but the image doesn't save as expected.
I've checked the error log file and it is generating the attach stack trace indicating the it is missing a file name. I've confirmed the image/name is fine by uploading to your demo site fine.
Any assistance would be appreciated,
Thanks.
bookstack.txt
@patoroco commented on GitHub (Feb 9, 2017):
I think that this was solved in last releases, isn't it @ssddanbrown?
@mrb2590 commented on GitHub (May 17, 2017):
I am having the same issue, I granted all users full permissions on the folder too and still getting this.
@mrb2590 commented on GitHub (May 17, 2017):
From the error log:
[2017-05-17 14:05:21] production.ERROR: exception 'ErrorException' with message 'file_get_contents(): Filename cannot be empty' in C:\inetpub\wwwroot\wiki.example.com\app\Services\ImageService.php:45
@ssddanbrown commented on GitHub (May 20, 2017):
@mrb2590 Any additional details would be great, Are you also on IIS? What version is you BookStack instance? What PHP version are you on?
@mrb2590 commented on GitHub (May 20, 2017):
@ssddanbrown Yup I am also on IIS. I can't check right now but I believe it is the latest version of BookStack I just installed it earlier this week and installed the release version. I am using php 5.6. I was looking into it earlier and saw in the ImageService class it is calling uploadedfile->getRealPath() into file_get_contents and getRealPath is returning false. The image file is in fact in the windows temp folder and did upload successfully but I guess saving it is where it's breaking. I can get more information to you on Monday. Hope this helps! Let me know what else you may need.
@RexSeven commented on GitHub (Jun 7, 2017):
I've had the same issue with image uploads on wordpress installs in IIS. To fix this you need to change the permissions on your temp folder. http://chris.wastedhalo.com/2011/01/wordpress-upload-permissions-on-iis-7-fix/
@mrb2590 commented on GitHub (Jun 7, 2017):
@RexSeven My hopes were up but unfortunately this did not work for me... It looks like inside app/Services/ImageService.php on line 46
$imageData = file_get_contents($uploadedFile->getRealPath());is failing because getRealPath is returning false. The file is in the temp folder though. If I replace That line with this:$imageData = file_get_contents($uploadedFile->getPathName());it does indeed work. So it's not a fix, but a workaround for now. I hate to touch the source code here to get this to work, but SplFileInfo::getRealPath() seems to be failing for some reason.@Abijeet commented on GitHub (Jun 13, 2017):
@mrb2590 - Same happened with me and I had to touch the source code.
@Abijeet commented on GitHub (Jun 13, 2017):
Another user reported an issue here - https://bugs.php.net/bug.php?id=74411&edit=1
@ssddanbrown commented on GitHub (May 12, 2018):
Thank you for opening this issue. As part of an issue clean-up we're closing off older support issues that may be outdated or irrelevant so therefore I am closing this issue.
If the issue remains and it is one you are still seeking to actively solve please ensure it is still not apparent on the latest version of BookStack then open up a new issue referencing this one.
Sorry if we were not able to solve your original request.
@LloydVillegas commented on GitHub (Mar 11, 2019):
Add permission for your IUSR in Windows temp folder