-
Notifications
You must be signed in to change notification settings - Fork 7.4k
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
player.tech().vhs is undefined on Safari #8654
Comments
👋 Thanks for opening your first issue here! 👋 If you're reporting a 🐞 bug, please make sure you include steps to reproduce it. We get a lot of issues on this repo, so please be patient and we will get back to you as soon as we can. |
By default Video.js prefers native HLS playback in Safari. If you want/need vhs features in Safari, set the vhs |
@phloxic Awesome, this seems to work great, thanks! |
@phloxic Reopening this one, as it does not seem on mobile Safari. Is there any way to make this work on mobile Safari please? |
See @mister-ben's answer here. It should work on iPad OS, see the VHS docs. |
@phloxic Thanks. Would you know of any other way to presign the URL on iPhones? Thanks! |
@vaclavgreif - I haven't got a clue ;-) Sorry. |
Description
I'm using Amazon S3 as a storage for my videos, and because the medias are not public, I need to presign the URLs to get access to the files. I'm using the following on the
onReady
hook:This works great everywhere except on Safari, where I get
player.tech().vhs is undefined
.When I set the media to public, the video plays fine even on Safari, but I need to handle the presigning. Is there a way to make this work on Safari?
Reduced test case
https://zitlehce.cz/protected-video/trailer-rosta-vaclavek/
Steps to reproduce
Errors
TypeError: undefined is not an object (evaluating 'g.current.tech().vhs.xhr')
What version of Video.js are you using?
8.10.2
Video.js plugins used.
No response
What browser(s) including version(s) does this occur with?
Safari
What OS(es) and version(s) does this occur with?
MacOS
The text was updated successfully, but these errors were encountered: