Setting liveSyncDuration & liveMaxLatencyDuration breaks player if you Stop then Play

[MasterEvilAce]

Browser: Chrome 64.0.3282.167
OS: Windows 10 x64
Clappr Version: 0.2.86

Steps to reproduce:

• Use a LIVE playlist
• set liveSyncDuration and liveMaxLatencyDuration values
  Example:
  hlsjsConfig: {
  liveSyncDuration: 8,
  liveMaxLatencyDuration: 18
  },
• Play the stream (it works fine and as expected with above values)
• Stop the stream with UI button (at any point)
• Play the stream again (with UI button)
• Player breaks with error in console

This issue also happens with the player at http://cdn.clappr.io/ if the above criteria is met.

console error is:
hls.js:15328 Uncaught Error: Illegal hls.js config: don't mix up liveSyncDurationCount/liveMaxLatencyDurationCount and liveSyncDuration/liveMaxLatencyDuration
at new t (hls.js:15328)
at e._setup (hls.js:152)
at e.play (hls.js:354)
at e.play (container.js:301)
at e.clicked (poster.js:84)
at HTMLDivElement.r.proxy (zepto.js:1616)

I'm assuming Clappr is resetting values for liveSyncDurationCount+liveMaxLatencyDurationCount (notice the COUNT) upon attempting to restart stream, even though they should not be provided when using the NON-count values.

[andrefilimono]

I encountered the same problem.

Comment for devs:
I think that you need to pass a copy of the config to HLS.js as it overrides config with default values:
https://github.com/clappr/clappr/blob/master/src/playbacks/hls/hls.js#L152

Something like this:
this._hls = new HLSJS(Object.assign({}, this.options.playback.hlsjsConfig) || {})

[andrefilimono]

Fixed in #1575