Fix HTTP Range handling for file responses (RFC 7233–compliance and Safari compatibility)

[giuscris]

This pull request refines the handling of HTTP file responses, particularly for range requests and HEAD requests in the FileResponse class. The changes improve standards compliance and ensure correct header and content length handling for various request scenarios.

Improvements to HTTP range and HEAD request handling:

• Refactored the logic in prepare() to always set the Accept-Ranges header for GET and HEAD requests, regardless of whether it was previously set.
• Improved parsing and handling of the Range header by ensuring range logic is only applied to GET requests and that the Content-Length header is set to 0 for unsatisfiable ranges. [1] [2]
• Moved the logic for setting $this->length = 0 for HEAD requests and empty content to the end of the method, ensuring that content length is always correct for these cases.

File streaming optimization:

• Updated the file streaming loop to avoid reading more bytes than necessary by adjusting the chunk size to the remaining length, preventing over-reading.

[Copilot]

Pull request overview

This PR refactors HTTP Range request handling in the FileResponse class to improve RFC 7233 compliance and optimize file streaming. The changes restructure the prepare() method to always set the Accept-Ranges header for GET and HEAD requests, move Range header processing logic before HEAD request handling, and optimize the file streaming loop to avoid reading excess bytes.

Key changes:

• Restructured Accept-Ranges header logic to always set it for GET/HEAD requests (if not already present)
• Moved Range header parsing and processing from after HEAD request early-return to a unified block that processes both GET and HEAD requests
• Optimized file streaming to read only the minimum of chunk size or remaining length

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.