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

isDynamic() returns always true even if MPD has changed its type from dynamic to static #3443

Closed
5 tasks done
tomraut opened this issue Nov 2, 2020 · 4 comments
Closed
5 tasks done
Milestone

Comments

@tomraut
Copy link

tomraut commented Nov 2, 2020

Environment
  • Link to playable MPD file: -
  • Dash.js version: 3.1.3
  • Browser name/version: Chrome 86.0.4240.111
  • OS name/version: OSX Mojave 10.14.6
Steps to reproduce

Start to play live stream and at some point update MPD's type from dynamic into static.

Observed behaviour

Player keeps playing stream fine to very end but it is not possible to detect if stream is still live or ondemand as calling for isDynamic() method returns always "true". Also, suggestion from @dsilhavy in Slack to listen for DYNAMIC_STREAM_COMPLETED event does not work as that event is described to be internal one and not being exposed to MediaPlayer level.

@tomraut
Copy link
Author

tomraut commented Nov 2, 2020

Some more information: Also this suggested way of hooking up to this event didn't work:

player.on('dynamicStreamCompleted',() => {
    console.log('completed');
})

I even tried adding breakpoint to ManifestUpdater where "eventBus.trigger(Events.DYNAMIC_STREAM_COMPLETED)" is executed and that breakpoint did hit when MPD's type changed to static, but event listeners didn't.

@bbert
Copy link
Contributor

bbert commented Nov 2, 2020

DYNAMIC_STREAM_COMPLETED is not (yet) public. See issue #3383 As proposed we can make it public and also rename it (DYNAMIC_TO_STATIC) as I suggested

@dsilhavy
Copy link
Collaborator

dsilhavy commented Nov 4, 2020

@tomraut #3444 introduced the media player event DYNAMIC_TO_STATIC. Please test and close this issue if it solves your problem

@dsilhavy dsilhavy added this to the 3.2.0 milestone Nov 4, 2020
@tomraut
Copy link
Author

tomraut commented Nov 4, 2020

@dsilhavy I've tested that one yesterday and can confirm that it solved issue.

@tomraut tomraut closed this as completed Nov 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants