video.js icon indicating copy to clipboard operation
video.js copied to clipboard

Player stuck after quality change and seek in rapid succession

Open Vox94 opened this issue 4 years ago • 12 comments

Description

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

Reduced test case !NB - I ended up copy-pasting the ENTIRE contrib quality levels in the file, as for some reason using a script src tag did not work for me. However, the contrib quality levels that is pasted in there is UNMODIFIED

When seeking backwards, if the video hasn't buffered enough, we ender an infinite loop.

Steps to reproduce

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

  1. Go to some moment on video (e.g. 70%)
  2. Switch to some quality (e.g. 1080p) (wait for first frame to load)
  3. Instantly switch to other quality (e.g. 240p)
  4. Instantly move progress bar to some earlier point in the video (e.g. 30%)

Notes: The video resumes if you go back to the video at the moment of the quality switch (in point 1), or any other time after that point (in the case of point 1 - 70% or after)

Error output

No errors are given whatsoever

Additional Information

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

versions

videojs

Tested on some versions > 7.10 (including latest, 7.11.5)

browsers

what browser are affected? Google chrome v88.0.4324.182 Mozilla Firefox v85.0.2

OSes

what platforms (operating systems and devices) are affected? Tested in Win10 - version 10.0.19041 Build 19041 (I have not checked in on phones, as it would be marginally difficult to fulfill the same steps there within such a short time period)

plugins

are any videojs plugins being used on the page? If so, please list them below. `videojs-contrib-quality-levels version 2.0.9 videojs-hls-quality-selector version 1.1.4

Vox94 avatar Feb 23 '21 13:02 Vox94

👋 Thanks for opening your first issue here! 👋

If you're reporting a 🐞 bug, please make sure you include steps to reproduce it. We get a lot of issues on this repo, so please be patient and we will get back to you as soon as we can. To help make it easier for us to investigate your issue, please follow the contributing guidelines.

welcome[bot] avatar Feb 23 '21 13:02 welcome[bot]

hey guys. any news here?

AlekseyChepurko avatar Jun 23 '21 15:06 AlekseyChepurko

I'm unable to reproduce. It's currently loading 7.12.3.

gkatsev avatar Jun 24 '21 19:06 gkatsev

Indeed. I do seem to no longer be able to reproduce it with the steps I provided in 7.12.3. However, switching to a different moment in the video, changing the resolution, switching again and changing again in rapid succession (basically repeated that for 5-6 times, as fast as I could) does indeed seem to eventually reproduce it. I'm attaching a visual with the case. OMf5dYZ7q9 https://imgur.com/a/pbNpOY0 (the second one is a link, as it is a bit too large to upload to github. I also let it run for a bit longer, to show that it's not a matter of waiting*)

*Currently still stuck, I've left it for some 15-20-ish minutes.

Vox94 avatar Jun 25 '21 12:06 Vox94

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Jan 09 '22 16:01 stale[bot]

This should still be active imo, just waiting for a reply.

Vox94 avatar Feb 14 '22 08:02 Vox94

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Apr 28 '22 04:04 stale[bot]

Still waiting

Vox94 avatar May 03 '22 13:05 Vox94

Same here, please don't close this ticket

privaloops avatar Jun 21 '22 13:06 privaloops

same here : https://github.com/videojs/http-streaming/issues/1273

privaloops avatar Jun 21 '22 13:06 privaloops

Hello, any news ?

privaloops avatar Aug 22 '22 06:08 privaloops

@privaloops were you able to solve this?

therealabdi2 avatar Nov 30 '23 06:11 therealabdi2