Skip to content

Conversation

@Sztig
Copy link
Contributor

@Sztig Sztig commented Jan 2, 2026

🎫 Issue IBX-11131

Description:

Apache using php-fpm is causing issues when streaming content on chromium based browsers.

I've changed the response to 206 even when providing full files.

Copy link
Contributor

@Steveb-p Steveb-p left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can confirm that this is expected behavior based on what I've found on the web.

See Partial request responses

There are three relevant statuses, when working with range requests:

A successful range request elicits a 206 Partial Content status from the server.
A range request that is out of bounds will result in a 416 Requested Range Not Satisfiable status, meaning that none of the range values overlap the extent of the resource. For example, the first-byte-pos of every range might be greater than the resource length.
If range requests are not supported, an 200 OK status is sent back and the entire response body is transmitted.

So I agree, even for full-length response including whole file, if the client asks for a range, we should respond with 206 unless going out of bounds.

@Steveb-p Steveb-p added Bug Something isn't working Ready for review labels Jan 4, 2026
@tomaszszopinski tomaszszopinski force-pushed the IBX-11131-apache-streaming-fix branch from 248b9e2 to 4ed512c Compare January 22, 2026 07:11
@sonarqubecloud
Copy link

Copy link

@tomaszszopinski tomaszszopinski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

QA approved on Ibexa DXP 4.6 ddev/apache-fpm

@alongosz alongosz changed the title IBX-11131: Streaming files fix for apache + php-fpm IBX-11131: Fixed streaming files for apache + php-fpm Jan 22, 2026
@alongosz alongosz merged commit ecd91f9 into 4.6 Jan 22, 2026
28 checks passed
@alongosz alongosz deleted the IBX-11131-apache-streaming-fix branch January 22, 2026 08:53
@Sztig
Copy link
Contributor Author

Sztig commented Jan 22, 2026

merged up a2f6335

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug Something isn't working QA approved

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants