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

HLS playback broken in v2.6.0 #1076

Closed
DispatchCommit opened this issue Feb 19, 2021 · 12 comments
Closed

HLS playback broken in v2.6.0 #1076

DispatchCommit opened this issue Feb 19, 2021 · 12 comments

Comments

@DispatchCommit
Copy link

Description

HLS video does not play with http-streaming v2.6.0 - There is no issue when using 2.5.0.

Sources

don't have a publicly accessible link available - could provide a temporary reproduction link if required though, would just take bypassing auth and cors checks.

Steps to reproduce

Explain in detail the exact steps necessary to reproduce the issue.

  1. Use Video.js with http-streaming v2.6.0
  2. Load an HLS live stream into the player
  3. ...
  4. Stream will never load, blank / black player with loading circle is all that appears. Does not go away.

Results

Expected

HLS Video to play like normal.

Error output

image

image

Additional Information

Myself and another developer for 2 separate live streaming platforms have both encountered this breaking issue, yet there are no existing reports of issues on Github.

image

videojs-http-streaming version

videojs-http-streaming 2.6.0

videojs version

video.js 7.11.4

Browsers

Chrome / Chromium based browsers.

Platforms

Probably all, but tested on Windows 10

Other Plugins

Contrib Quality Levels, HLS quality selector

Other JavaScript

I use Vue.js inside Nuxt.js. The other developer is using React. Both appear to have issues.


Further investigation needed likely, but I wanted to make sure I brought this to attention first and created a place to document issues, and reproducible instructions.

@gkatsev
Copy link
Member

gkatsev commented Feb 19, 2021

Does it work on https://videojs-http-streaming.netlify.app/ or still broken there?

@gkatsev
Copy link
Member

gkatsev commented Feb 19, 2021

Reading the convo, yeah, I think it likely is fixed by #1072. It's broken if you're minifying your build (which you probably are).
We're going to release a patch tomorrow with that change but if you can try linking in the main branch and see whether it works for you, that would be helpful.

@DispatchCommit
Copy link
Author

It's broken if you're minifying your build

Well of course minifying files! 😄 and the error message looks like an issue caused by minification.

ALSO, I accidentally pushed v2.6.0 into our production build thinking it was fine because running in the dev build (un-minified) worked fine. Wasn't until I verified the deployment did I realize that the player broke.

We're going to release a patch tomorrow with that change

Awesome, based on what we've discussed so far, I think this would resolve the issue.

try linking in the main branch and see whether it works for you, that would be helpful.

Oh, gotta remember how to do that through NPM... believe it's:

npm install videojs/http-streaming#main

(if that works, I'll push to staging and let you know if it works shortly)

@gkatsev
Copy link
Member

gkatsev commented Feb 19, 2021

Unfortunately, installing from github won't work because it doesn't have the built files.

@gkatsev
Copy link
Member

gkatsev commented Feb 19, 2021

ALSO, I accidentally pushed v2.6.0 into our production build thinking it was fine because running in the dev build (un-minified) worked fine. Wasn't until I verified the deployment did I realize that the player broke.

Oops, sorry that happened.

@DispatchCommit
Copy link
Author

installing from github won't work

That's a shame, I love that trick for quick testing and feedback without much effort 😞
It's 9pm for me, and if the patch ships tomorrow, would be easiest for me to just wait for the patch and test that.

Oops, sorry that happened.

😄 All good, that's why rollbacks exist. At least this issue was immediately obvious.

@gkatsev
Copy link
Member

gkatsev commented Feb 19, 2021

👍🏻 sounds good.

@gkatsev
Copy link
Member

gkatsev commented Feb 19, 2021

@DispatchCommit VHS 2.6.1 is out. Please try it out!

@datagutt
Copy link

VHS 2.6.1 seems to work fine on my end (i am the person in the screenshot).
We should probably hear from @DispatchCommit first though, to see if it works on his platform as well.

@gkatsev
Copy link
Member

gkatsev commented Feb 19, 2021

Thanks!

@DispatchCommit
Copy link
Author

Confirming this issue is resolved in VHS v2.6.1

Thanks for the help @gkatsev, glad to see this working as expected!

@gkatsev
Copy link
Member

gkatsev commented Feb 23, 2021

Thanks for circling back!

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

No branches or pull requests

3 participants