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

Hls.js displays wrong duration in the video player #3976

Closed
5 tasks done
iyromanov opened this issue May 31, 2021 · 5 comments · Fixed by #3997
Closed
5 tasks done

Hls.js displays wrong duration in the video player #3976

iyromanov opened this issue May 31, 2021 · 5 comments · Fixed by #3997
Milestone

Comments

@iyromanov
Copy link

What version of Hls.js are you using?

1.0.4

What browser and OS (including versions) are you using?

macOS Catalina 10.15.7
Chrome 91.0.4472.77

Test stream:

https://api-http.aitube.kz/kz.aitudala.aitube.staticaccess/video/205121f5-16ef-47a3-80ca-b356a73b56d0/video

Configuration:

Default config on demo page:

{
  "debug": true,
  "enableWorker": true,
  "lowLatencyMode": true,
  "backBufferLength": 90
}

Checklist

  • The stream has correct Access-Control-Allow-Origin headers (CORS)
  • There are no network errors such as 404s in the browser console when trying to play the stream

Steps to reproduce

Open https://hls-js-dev.netlify.com/demo and put https://api-http.aitube.kz/kz.aitudala.aitube.staticaccess/video/205121f5-16ef-47a3-80ca-b356a73b56d0/video

Expected behavior

Duration of the video must be 00:49

Actual behavior

Duration of the video is 54:06:27

Console output

Nothing special

@vlazh
Copy link

vlazh commented Jun 1, 2021

Same issue. First fragment has very big appendedPTS.
Works fine with hls.js@1.0.3.

@robwalch
Copy link
Collaborator

robwalch commented Jun 3, 2021

Somethings up with your audio. Take a look at remuxAudio; what has changed there; why is the audio appended longer than 4-5 seconds? Set "enableWorker": false, to debug the remuxer.

https://github.com/video-dev/hls.js/blob/v1.0.5/src/remux/mp4-remuxer.ts#L795-L821

@robwalch
Copy link
Collaborator

robwalch commented Jun 3, 2021

@iyromanov and @vlazh please take a look at #3997 demo build https://deploy-preview-3997--hls-js-dev.netlify.app/demo/ and let me know if this addresses the issue or you see any other AAC related regressions. Thanks!

@robwalch robwalch added this to the 1.0.6 milestone Jun 3, 2021
@vlazh
Copy link

vlazh commented Jun 6, 2021

@robwalch v1.0.6 works fine for me. Thank you!

@iyromanov
Copy link
Author

@robwalch now works as expected, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants