The Wayback Machine - https://web.archive.org/web/20200526065429/https://github.com/videojs/video.js/issues/5932
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

Ability to override VHS? #5932

Open
stevendesu opened this issue Apr 11, 2019 · 3 comments
Open

Ability to override VHS? #5932

stevendesu opened this issue Apr 11, 2019 · 3 comments

Comments

@stevendesu
Copy link

@stevendesu stevendesu commented Apr 11, 2019

Description

Briefly describe the issue.
Include a reduced test case, we have a starter template on JSBin you can use.

I've run into a few issues with VideoJS HTTP Streaming:

I don't encounter either of these issues when playing the same content on an older VideoJS version using HLS.js

I would like the ability to override VHS as the default handler for HLS videos and instead use HLS.js in my application

Steps to reproduce

See the two linked issues

Results

Expected

Videos all play without issue

Actual

Some videos do not play properly (see linked issues)

Error output

See linked issues

Additional Information

Please include any additional information necessary here. Including the following:

versions

videojs

what version of videojs does this occur with?
v7.4.1

browsers

what browser are affected?

  • Firefox 66
  • Chrome 72
  • Probably all others, but not tested

OSes

what platforms (operating systems and devices) are affected?

  • Mac OS X 10.14
  • Windows 10
  • Probably all others, but not tested

plugins

are any videojs plugins being used on the page? If so, please list them below.
Only videojs-http-streaming (included with VideoJS)

@mister-ben
Copy link
Contributor

@mister-ben mister-ben commented Apr 12, 2019

There's a "core" version without VHS, video.core.min.js

@stevendesu
Copy link
Author

@stevendesu stevendesu commented Apr 12, 2019

@mister-ben That sounds perfect. Thank you for the update. Could the README maybe be updated to mention the "core" version for anyone else looking for similar functionality?

@gkatsev
Copy link
Member

@gkatsev gkatsev commented Apr 16, 2019

There is the slightly issue with #5223, the current workaround is to alias video.js to be video.js/core for all dependencies, via webpack or other bundler.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
3 participants
You can’t perform that action at this time.