Support new Periods adding new encryption info

[TheJohnBowers]

Have you read the FAQ and checked for duplicate open issues?:
Yes, I didn't see anything that looked like a duplicate issue

What version of Shaka Player are you using?:
2.3 built from master

Can you reproduce the issue with our latest release version?:
It also occurs when building 2.3 from the v2.3.x branch

Can you reproduce the issue with the latest code from master?:
Yes

Are you using the demo app or your own custom app?:
Demo App

If custom app, can you reproduce the issue using our demo app?:
NA

What browser and OS are you using?:
Chrome 64 on Mac OS -- Also Firefox Quantum 58.02 on MacOS

What are the manifest and license server URIs?:

{
  name: 'Downlynk Dash Live with Ad Breaks - duseqa1',
  manifestUri: 'https://content-duseqa1.downlynk.com/channel/29e9c85c070f413fb71a3b2a39eed208.mpd?ad=dashad', // eslint-disable-line max-len
  encoder: shakaAssets.Encoder.UPLYNK,
  source: shakaAssets.Source.UPLYNK,
  drm: [
    shakaAssets.KeySystem.WIDEVINE,
    shakaAssets.KeySystem.PLAYREADY
  ],
  features: [
    shakaAssets.Feature.MP4,
    shakaAssets.Feature.PSSH,
    shakaAssets.Feature.MULTIKEY,
    shakaAssets.Feature.HIGH_DEFINITION
  ],
  licenseServers: {
    'com.microsoft.playready': 'https://content-duseqa1.downlynk.com/pr',
    'com.widevine.alpha': 'https://content-duseqa1.downlynk.com/wv'
  },
  requestFilter: shakaAssets.UplynkRequestFilter,
  responseFilter: shakaAssets.UplynkResponseFilter
},

What did you do? / What did you expect to happen? / What actually happened?
The above test stream is running a sample ad that is 116 seconds in length every 5 minutes. If the live stream is joined during the drm protected content portion of the stream, everything works fine. Playback transitions into and out of the unencrypted ad portion perfectly over and over again.

IF however I just happen to join the stream during the ad portion when the manifest contains no drm protected period, playback stalls when a period eventually pops into the live window with widevine content protection. Playback seems to proceed to the end of the unencrypted period, and then hang. A few segments of the new period seem to be downloaded an buffered, but playback doesn't proceed.

Firefox actually gives a nasty decoder error and stops, while chrome just kind of sits there. Here is the firefox error:

Shaka Error MEDIA.VIDEO_ERROR (3,,NS_ERROR_DOM_MEDIA_DECODE_ERR (0x806e0004) - mozilla::MediaResult mozilla::AppleATDecoder::DecodeSample(mozilla::MediaRawData *): Error decoding audio sample: 1852797029 @ 117269001)

The network pane doesn't show any attempt to request the widevine license when the protected content shows up in the live window, or at any other time during playback in this scenario. If I reload the stream once the live window again advertises protected content, playback works fine.

This is an important scenario to work if someone wants to use unencrypted/unprotected content for ads as you cannot be sure when viewers will join a live event.

I am not seeing this issue with ExoPlayer

[chrisfillmore]

This looks similar to #1094.

[TheJohnBowers]

It looks exactly like #1094 actually. I hadn't noticed that issue as it was closed. If I read that issue correctly, a fix was applied to setup the EME if license server config suggested that protected content would be provided. I do have license servers setup in the player config, so the provided fix should fix my issue? The EME log does show some activity when I start during an unencrypted ad period, but still no license is requested when the protected period shows up. I update master just this morning to ensure I had everything. So I have the code that is supposed to provide the fix, yet this issue persists.

eme_start_on_drm_protected_asset.log
eme_start_on_unencrypted_asset.log

[TheJohnBowers]

Just checked out the v2.3.x branch and verified the same issue occurs there also.

Perhaps any new license is a problem. I create a new test live stream that starts with encrypted content and then every 3 minutes a new period with new encryption keys starts. This would require a new license to be requested. I am watching the network pane, and I am sure that never happens. Because of this playback dies at the period boundary. It basically looks the same as when the live stream starts on a unencrypted period. So perhaps this has nothing specifically to do with starting on a unencrypted period, just a failure to request new licenses as they are required.

{
  name: 'Live New Asset / New Key Every 3 Min',
  manifestUri: 'https://content-duseqa1.downlynk.com/channel/40b7f8553f6a41a7a1c00b0147b5618a.mpd', // eslint-disable-line max-len
  encoder: shakaAssets.Encoder.UPLYNK,
  source: shakaAssets.Source.UPLYNK,
  drm: [
    shakaAssets.KeySystem.WIDEVINE,
    shakaAssets.KeySystem.PLAYREADY
  ],
  features: [
    shakaAssets.Feature.MP4,
    shakaAssets.Feature.PSSH,
    shakaAssets.Feature.MULTIKEY,
    shakaAssets.Feature.HIGH_DEFINITION
  ],
  licenseServers: {
    'com.microsoft.playready': 'https://content-duseqa1.downlynk.com/pr',
    'com.widevine.alpha': 'https://content-duseqa1.downlynk.com/wv'
  },
  requestFilter: shakaAssets.UplynkRequestFilter,
  responseFilter: shakaAssets.UplynkResponseFilter
},

[TheModMaker]

The problem is that we currently only support key rotation when there are no pssh data in the manifest. See #1107 for some more info. The way we support key rotation is by listening for encrypted events from the browser when it sees a pssh box in the media. We use this to send the license request. When the pssh data is in the manifest, we only use that and ignore encrypted events. But right now, we don't look for new pssh data in the new Periods. So when we get a new Period, we should look for new pssh data and handle them.

[chrisfillmore]

@TheModMaker is this ticket a duplicate of #1107 now?

[TheJohnBowers]

The PSSH data on these streams only exists in the manifest, and not in the init segments at all. So in that sense it is slightly different than what seems to be describe in #1107. Taking the PSSH data out of the manifest wouldn't solve anything as appears to be the case with 1107. Other than that, it does look like it would be a very similar issue.

[joeyparrish]

We will look into supporting new init data from manifest updates. Sorry for the trouble!

[TheJohnBowers]

Is this on the radar anywhere? Scheduled to be looked at for any particular targeted release, etc?

[TheModMaker]

@TheJohnBowers This is scheduled for v2.5, which will be our next feature release. We will be working on this soon, but no one is working on it right now.