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

Quirk in Safari: duration: infinite #657

Closed
golddranks opened this issue Nov 25, 2016 · 3 comments · Fixed by #658
Closed

Quirk in Safari: duration: infinite #657

golddranks opened this issue Nov 25, 2016 · 3 comments · Fixed by #658

Comments

@golddranks
Copy link
Contributor

golddranks commented Nov 25, 2016

There is a quirk in Safari explained in detail here(1): if the server doesn't respond correctly to the range header Safari sends, it sets the duration to infinite. I was bitten by this, as I serve audio from a custom backend to howler. Even if the audio isn't infinite, howler's end event will never run. I can fix my backend and will do so, but it would be great (and indeed, in line with its aspiration to support edge cases) if howler also defensively supported firing the end even with seemingly infinite-duration audio. I peeked the code quickly and it seems that the end event is currently only fired in accordance with the duration. I think that subscribing to the DOM ended event in case of infinite html5 audio will help to fix this edge case.

(1) http://stackoverflow.com/questions/9629223/audio-duration-returns-infinity-on-safari-when-mp3-is-served-from-php

@goldfire
Copy link
Owner

Closing due to inactivity. If this is still an issue, reply with more info and I'll reopen.

@golddranks
Copy link
Contributor Author

golddranks commented Jun 21, 2020

Closing this by inactivity is a non-solution.

@golddranks
Copy link
Contributor Author

golddranks commented Jun 21, 2020

#658 ← The pull request that fixes this issue is still open, and not merged. Instead of closing this, please consider merging the PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants