mirror of
https://github.com/BookStackApp/BookStack.git
synced 2026-02-10 19:06:16 +03:00
Bookstack Attachment Update failing to parse "name" from form-data #3392
Reference in New Issue
Block 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 @ikubetoomuzik on GitHub (Dec 7, 2022).
Describe the Bug
Unable to use the attachment update API. The response continually says " Undefined array key "name" at /app/www/app/Uploads/AttachmentService.php:163".
Steps to Reproduce
{ name: $file_name, uploaded_to: X, file: $file_data }
Expected Behaviour
Expect a response similar to:
{
"id": Y,
"name": $file_name,
"extension": "",
"uploaded_to": X,
"external": true,
"order": 2,
"created_by": 1,
"updated_by": 1,
"created_at": "2021-10-20T06:35:46.000000Z",
"updated_at": "2021-10-20T06:37:11.000000Z"
}
Screenshots or Additional Context
I tried enabling DEBUG_MODE in the application but the trace when making this request did not get any larger.
Browser Details
No response
Exact BookStack Version
v22.11
PHP Version
8.0
Hosting Environment
Installed using official docker container on a Rocky Linux 9 host.
@ssddanbrown commented on GitHub (Dec 7, 2022):
Hi @ikubetoomuzik,
You might be hitting the issue reported here. Some context is in my linked comment. As per that comment, You could try using a POST request and provide a
_method: 'PUT'parameter. If that does not work, it would help to have some insight into the code being used to send this request.@ikubetoomuzik commented on GitHub (Dec 7, 2022):
Hello @ssddanbrown
Thank-you so much for the quick response. Adding _method=PUT did get things working correctly again. Sorry to bother with something that is already reported, I am not great with searching through GitHub issues.
@ssddanbrown commented on GitHub (Dec 7, 2022):
Happy that worked, No worries at all, It's an awkward issue that shouldn't really be there TBH but I've been split on approaches to fixing it. I might need to document the workaround until I land on a solution.
@ikubetoomuzik commented on GitHub (Dec 7, 2022):
Yeah a small comment in the API docs would be perfect. I don't have much in the way of PHP knowledge but if you have a solution you want to try I have some free time in the evenings to attempt implementing/testing it myself.