Skip to content
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

🐛 Bug Report: Storage download file - Bad Range header handling #3706

Open
2 tasks done
starkdmi opened this issue Aug 21, 2022 · 1 comment
Open
2 tasks done

🐛 Bug Report: Storage download file - Bad Range header handling #3706

starkdmi opened this issue Aug 21, 2022 · 1 comment
Labels
bug Something isn't working product / storage Fixes and upgrades for the Appwrite Storage.

Comments

@starkdmi
Copy link

starkdmi commented Aug 21, 2022

👟 Reproduction steps

Download request at /storage/buckets/bucketId/files/fileId/download with header "Range: bytes=0-" or "Range: bytes=START-". Postman and file bigger than 5MB used for test.

👍 Expected behavior

Expected to get full file same as it works for header "Range: bytes=0-FILESIZE" or without Range header at all.

👎 Actual Behavior

Only chunk (2MB of size) returned. Content-Length header is missing.

🎲 Appwrite version

Version 0.15.x

💻 Operating system

Linux

🧱 Your Environment

No response

👀 Have you spent some time to check if this issue has been raised before?

  • I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

@starkdmi starkdmi added the bug Something isn't working label Aug 21, 2022
@starkdmi
Copy link
Author

According to tests https://github.com/appwrite/appwrite/blob/master/tests/e2e/Services/Storage/StorageBase.php#L354 range "bytes 0-" is invalid (comments, maybe mistake) but pass 206 status code testing - file size should be tested as well.

@Meldiron Meldiron added the product / storage Fixes and upgrades for the Appwrite Storage. label Aug 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working product / storage Fixes and upgrades for the Appwrite Storage.
Projects
None yet
Development

No branches or pull requests

2 participants