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

IMA Extension : Multiple Ads not played with empty vast #8205

Closed
hyl0376 opened this issue Nov 11, 2020 · 3 comments
Closed

IMA Extension : Multiple Ads not played with empty vast #8205

hyl0376 opened this issue Nov 11, 2020 · 3 comments
Assignees
Labels

Comments

@hyl0376
Copy link

hyl0376 commented Nov 11, 2020

[REQUIRED] Issue description

Our VMAP AD contains 2 pre-rolls, the second of which can be empty. When I try to play this adTagUrl with demo app, the player plays the first ad well but it freezes before the second ad plays. There's no sound or buffering. I found #5006, #6097 but it still looks weird. This adTagUrl works fine in IMA SDK sample app (https://github.com/googleads/googleads-ima-android/releases AdvancedExample v3.24). I've emailed this adTagUrl to dev.exoplayer@gmail.com.

[REQUIRED] Reproduction steps

I just switch "ad_tag_uri" in media.exolist.json to our custom adTagUrl and play.

[REQUIRED] Link to test content

Our custom adTagUrl has been sent to dev.exoplayer@gmail.com.

[REQUIRED] A full bug report captured from the device

A logcat and bug report have been sent to email too.

[REQUIRED] Version of ExoPlayer being used

2.12.1

[REQUIRED] Device(s) and version(s) of Android being used

Google Pixel 3 Android 11, API 30

@hyl0376

This comment has been minimized.

@andrewlewis
Copy link
Collaborator

Sorry for the delay. I don't see any events from the IMA SDK relating to the second ad in the preroll ad pod, which is unexpected. I'll check the AdvancedExample to find out why it's working there, and then see if we can at least time out the ad loading to avoid playback getting stuck. Please let me know if you've found out anything else about what's going on since filing this.

@hyl0376
Copy link
Author

hyl0376 commented Nov 30, 2020

Thanks for your kind support. It works fine on exoplayer 2.11.4(which I have used before 2.12.1 update) so I guess it seems to be due to preloading ads.
I'll continue to dig into the problem, too, so let me know if there is anything new.

ojw28 pushed a commit that referenced this issue Dec 3, 2020
Previously `MediaPeriodQueue` would return null if an ad media URI hadn't
loaded yet, but this meant that the player could be stuck in `STATE_READY` if
an `AdsLoader` unexpectedly didn't provide an ad URI. Fix this behavior by
masking ad media periods. `MaskingMediaPeriod` no longer requires a
`MediaSource` to instantiate it.

This also fixes a specific case where playback gets stuck when using the IMA
extension with an empty ad where the IMA SDK unexpectedly doesn't notify the ad
group fetch error.

Issue: #8205
PiperOrigin-RevId: 344984824
icbaker pushed a commit that referenced this issue Jan 11, 2021
Previously `MediaPeriodQueue` would return null if an ad media URI hadn't
loaded yet, but this meant that the player could be stuck in `STATE_READY` if
an `AdsLoader` unexpectedly didn't provide an ad URI. Fix this behavior by
masking ad media periods. `MaskingMediaPeriod` no longer requires a
`MediaSource` to instantiate it.

This also fixes a specific case where playback gets stuck when using the IMA
extension with an empty ad where the IMA SDK unexpectedly doesn't notify the ad
group fetch error.

Issue: #8205
PiperOrigin-RevId: 344984824
@google google locked and limited conversation to collaborators Mar 22, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants