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

WAV Files and Large Videos and new Safari Request Range headers #485

Open
DiegoPino opened this issue Dec 6, 2024 · 1 comment
Open
Assignees
Labels
external bug It is not us, it is them File and HTTP
Milestone

Comments

@DiegoPino
Copy link
Member

What?

Well I should be sleeping. But was testing some 1+ Gigabytes WAV in Safari and they did not play. But they Play on Chrome and Mozilla. Why? Bc Safari request them via a Range Request from 0 bytes to Filesize -1 ... (basically the whole thing) and PHP/Symfony/Drupal, under that condition assumes, well... you are asking for the whole file! No range for you, here is a 200 RESPONSE with no range ... (bc that is kinda obvious?) instead of a 206 with range the same as the requested ...

In this case I will give it to Symfony. The response is correct, but Safari is used in a LOT of places... so I will have to code my way around

@DiegoPino DiegoPino added external bug It is not us, it is them File and HTTP labels Dec 6, 2024
@DiegoPino DiegoPino added this to the 1.5.0 milestone Dec 6, 2024
@DiegoPino DiegoPino self-assigned this Dec 6, 2024
DiegoPino added a commit that referenced this issue Dec 6, 2024
Because IOS/Safari as dumb (very) see #485

also do some indentation...
@DiegoPino
Copy link
Member Author

327f18b does the trick. I need to wrap up that larger pull and merge bc it is become almost a complete release in a single one

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
external bug It is not us, it is them File and HTTP
Projects
None yet
Development

No branches or pull requests

1 participant