Video load fails upon redirect [#1482606]

Comment #1

Jorrit CreditAttribution: Jorrit commented 18 March 2012 at 13:21

Status:

Active

» Postponed (maintainer needs more info)

Can I see a webpage where Video.js fails because of this problem?

Log in or register to post comments

Comment #2

dietric@gmail.com

he/him

German

CreditAttribution: dietric@gmail.com commented 19 March 2012 at 14:36

I don't have anything in a environment at this point that is publicly accessible.
It should be easy to reproduce though by pointing a link field to "http://api.kickapps.com/rest/resource/VIDEO/1702801/30809?returnMP4=true"

Log in or register to post comments

Comment #3

Jorrit CreditAttribution: Jorrit commented 19 March 2012 at 14:52

Before I do that, what version of Video.js are you using (I mean the library you get from Videojs.com) and what browser are you using?

Log in or register to post comments

Comment #4

dietric@gmail.com

he/him

German

CreditAttribution: dietric@gmail.com commented 19 March 2012 at 15:48

Hi,
i am using Version 3.1.0 of VideoJS, and I have been using Google Chrome 17.0.963.78 m on Windows. I have also tested it with other browsers (Firefox, Safari) with the same results, I can provide version number if you think that would be helpful.
Thanks
Dietrich

Log in or register to post comments

Comment #5

Jorrit CreditAttribution: Jorrit commented 19 March 2012 at 21:43

There are two problems, none of them have to do with the redirect:

1) The module strips of the query string (?returnMP4=true), creating a URL that the player can't handle.
2) Because the link doesn't end in an extension, the module can't determine the mime type and defaults to video/webm (for historical reasons).

I am going to make two changes:

1) Fix the query string
2) Allow you to override the mime type using the title field
3) Set the default mime type to video/mp4 when using the link field, because that seems to be the most widely used one.

You can try these changes tomorrow, when the -dev release is refreshed.

Log in or register to post comments

Comment #6

dietric@gmail.com

he/him

German

CreditAttribution: dietric@gmail.com commented 20 March 2012 at 14:42

Thanks so much for doing that! I noticed the changes are present in videojs 7.x-2.x-dev. However, I am using 6.x-2.0-rc3 - are you planning to update that too?
Thanks
Dietrich

Log in or register to post comments

Comment #7

Jorrit CreditAttribution: Jorrit commented 20 March 2012 at 19:39

I hadn't noticed that, fixed it there as well.

Log in or register to post comments

Comment #8

dietric@gmail.com

he/him

German

CreditAttribution: dietric@gmail.com commented 21 March 2012 at 16:24

That seems to have fixed it for the URL I mentioned, but I still have a lot that seem to be returning different MIM/TYPES. I think that those don't even work with Video-JS outside Drupal, but I was curious if you ever encountered those before I dive any deeper into the actual Video JS library:
http://api.kickapps.com/rest/resource/VIDEO/905450/30809
http://api.kickapps.com/rest/resource/VIDEO/905476/30809
http://api.kickapps.com/rest/resource/VIDEO/905487/30809
Those return SMIL as the MIM type. SMIL files are used by some CDN networks as an additional layer to provide load balancing. The SMIL file contains the location information for the media file residing in one of the edge servers the CDN has.

Log in or register to post comments

Comment #9

Jorrit CreditAttribution: Jorrit commented 21 March 2012 at 17:38

The video.js library is just a wrapper on top of the HTML5 <video> element, which doesn't support smil. My suggestion is that you keep using ?returnMP4=true to get a URL to an actual video file.