Skip to content

Multiple mediafiles #73

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

Merged
merged 2 commits into from
Mar 23, 2023
Merged

Multiple mediafiles #73

merged 2 commits into from
Mar 23, 2023

Conversation

wabiloo
Copy link
Contributor

@wabiloo wabiloo commented Mar 3, 2023

MRSS files can now contain multiple URLs, and the VAST payload will return all of them as nodes.
This is useful to provide ad creatives in multiple versions (eg. HLS, DASH, MP4), allowing players / SSAI engines to select the most appropriate one for any use case.

There is somewhat of a gross assumption that width, height, codec and bitrate applies to all renditions of the ads, but I think that's OK in many cases and can be approved down the road

@birme birme requested review from Nfrederiksen March 6, 2023 15:14
Copy link
Contributor

@birme birme left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@birme birme merged commit 1c742c7 into Eyevinn:main Mar 23, 2023
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

Successfully merging this pull request may close these issues.

2 participants