-
-
Notifications
You must be signed in to change notification settings - Fork 80
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
File stored in S3-compatible storage can't be download or viewed in PHP 8.2 #536
Comments
Got the same issue. |
Same issue. |
Closed
1 task
This was referenced May 24, 2024
Open
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
System Info
Describe the bug
XBackBone returns 502 when downloading / view file as raw in PHP 8.2.
To Reproduce
Steps to reproduce the behavior:
nginx conf
XBackBone conf
Expected behavior
XBackBone behaves the same as PHP 7.4 / 8.0 / 8.1 which means you can download the file and view as raw.
Screenshots
None.
Logs
XBackBone logs (logs/log-2023-06-22.txt)
nginx logs (xbackbone.error.log)
Nothing useful returned in browser (
debug
is true in XBackBone conf):Quick tests against other PHP version
I've also tested on linuxserver/docker-nginx against different PHP version and the latest linuxserver/xbackbone
linuxserver/docker-nginx:1.20.2-r1-ls191
linuxserver/docker-nginx:1.20.2-r1-ls203
linuxserver/docker-nginx:1.22.1-r0-ls225
linuxserver/docker-nginx:1.24.0-r6-ls228
(latest)linuxserver/xbackbone:3.6.3-ls99
(latest)Dirty workaround
https://github.com/SergiX44/XBackBone/blob/0951638dc8e80667cbd9053bf1a6926470d7cc6e/app/Controllers/MediaController.php#L400-L407
This dirty hack works for the random txt file above I uploaded. Files I stored like doc, zip and pdf also works again.
I've also tried commenting out (one at a time) other set
Content-Length
header in L421, L480, L501. Only commenting L405 works in this case.The text was updated successfully, but these errors were encountered: