Skip to content
This repository has been archived by the owner on Dec 10, 2020. It is now read-only.

4.4.0 on CDN works. 4.4.1 in videojs download zip file broken #104

Closed
sethborg opened this issue Jun 19, 2014 · 3 comments
Closed

4.4.0 on CDN works. 4.4.1 in videojs download zip file broken #104

sethborg opened this issue Jun 19, 2014 · 3 comments

Comments

@sethborg
Copy link

  1. When hosting your example on jsbin it loads the video only once. If I copy that code and host it on a domain myself that isn't jsbin it loads the video 3 times! why??
  2. Something seems to be wrong with the self hosted version vs the CDN even though the video.js file is exactly the same. The CDN's http://vjs.zencdn.net/4.6/video-js.swf is 16,803 bytes while the one in the videojs 4.6.3 download is 16,899 bytes. Did someone not update it or did they not update the CDN version?

Referencing: videojs/video.js#1300

Here we use the files on: http://vjs.zencdn.net/4.6/ and the video loads 3 times in firefox and the big play button disappears as it should
http://sethb.org/videojstest/flash_cdn.htm

also the exact code from your example loads the video 3 times in firefox and the big play button disappears as it should. It doesn't load the video 3 times when hosted on jsbin.com. I noticed your example does not contain this line ( videojs.options.flash.swf = "http://vjs.zencdn.net/4.6/video-js.swf"; ) yet the flash still works. Why do self hosted ones need this but the CDN doesn't?
http://sethb.org/videojstest/flash_your_example.htm

Here we use the self hosted 4.6.3 files and the video loads 3 times in firefox but the big play button does not disappear
http://sethb.org/videojstest/flash_local.htm

The main issue is that the swf seems to be requesting the video 3 times AND in IE8 the 2 CDN examples above work but http://sethb.org/videojstest/flash_local.htm shows the big play button and no control bar at all.

@sethborg
Copy link
Author

Could someone please look into this? The player is broken until this is fixed.

@heff
Copy link
Member

heff commented Jul 11, 2014

In progress. #1343

@gkatsev
Copy link
Member

gkatsev commented Sep 26, 2014

Given that videojs/video.js#1343 and #106 have been fixed, I would assume this issue has been addressed.
Going to close it, but please let us know if I am mistaken and the there is still a problem.

@gkatsev gkatsev closed this as completed Sep 26, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants