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

MPD Patch does not work for multi-period MPDs #177

Closed
tobbee opened this issue Apr 21, 2024 · 0 comments · Fixed by #178
Closed

MPD Patch does not work for multi-period MPDs #177

tobbee opened this issue Apr 21, 2024 · 0 comments · Fixed by #178
Labels
bug Something isn't working

Comments

@tobbee
Copy link
Contributor

tobbee commented Apr 21, 2024

A multi-period source like http://localhost:8888/livesim2/segtimeline_1/patch_60/periods_60/testpic_2s/Manifest.mpd does not result in proper patches.

This can be seen by asking for some specific MPDs and a Patch like:

curl http://localhost:8888/livesim2/segtimeline_1/patch_60/periods_60/testpic_2s/Manifest.mpd?publishTime=2024-04-21T06:10:58Z > 1.mpd
curl http://localhost:8888/livesim2/segtimeline_1/patch_60/periods_60/testpic_2s/Manifest.mpd?publishTime=2024-04-21T06:11:04Z > 2.mpd
curl http://localhost:8888/patch/livesim2/segtimeline_1/patch_60/periods_60/testpic_2s/Manifest.mpd\?publishTime\=2024-04-21T06:10:58Z&nowDate=2024-04-21T06:11:04Z > 1-2.mpp

The patch will remove the first Period, but the update of later period is not proper including addressing like

<replace sel="/MPD/Period[3]/@id">P28561331</replace> <replace sel="/MPD/Period[3]/@start">PT476022H11M</replace>

Relates to #155 and #174

@tobbee tobbee added the bug Something isn't working label Apr 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant