Unable to seek on apple browsers

[rgaudin]

On all apple readers (macOS and iOS Kiwix) but also Safari for both OS, the video progress bar displays properly and moves along with playing but it's impossible to seek to a different position.

Clicking anywhere on the bar has same effect as clicking on current position: a quick pause/play.

We should test if this affects mp4 video as well (natively supported) and if not, it's most probably an issue with ogvjs.

[satyamtg]

@rgaudin I can confirm that this is an issue with ogvjs. We can actually reproduce this on other platforms than Apple by simply changing the techOrder": ["html5", "ogvjs"] to techOrder": ["ogvjs"] in the templates which will force videojs to use ogvjs to play videos. We also see errors in console saying that video is unseekable.

[rgaudin]

OK I'm leaving this open but categorizing it as awaiting-upstream.

[kelson42]

@rgaudin @satyamtg I would check again this with http://library.kiwix.org/mali-pour-les-nuls_fr_all_2020-04 because we have now there a kiwix-serve which should support byte-range requests properly.

[rgaudin]

No improvement using 3.1.1:

Error: Cannot seek a non-seekable stream

if (this._stream && !this._stream.seekable)
  throw new Error("Cannot seek a non-seekable stream");
--
seekable: {
  get: function get() {
    return this.demuxer.seekable
  }
},

The ogv.js readme: clearly states that this should work with Range header.

Actually, when testing the build folder of the zim file outside kiwix-serve, it seeks as expected.

It's hard to debug what happens with the devtools but it seems that it loads the video completely in one request (requests the full range) but still can't seek.

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be now be reviewed manually. Thank you for your contributions.

[rgaudin]

openzim/ted@e4f9cc4 fixes it. We can thus tweak our template to detect no-webm support and apply the same CSS tweaks.

[kelson42]

@satyamtg @rgaudin Seems quite important to rapidly fix this on this scraper as well and make a new release.

[kelson42]

@satyamtg Would you be able please to fix that one like you fixed TED. We would then have to make a new release of the youtube2zim.

[satyamtg]

@satyamtg Would you be able please to fix that one like you fixed TED. We would then have to make a new release of the youtube2zim.

If you are talking of this issue, I have already added the fix in #119. Waiting for a review from @rgaudin. I don't have an apple device, so can't really test.