PLAT-10838: add entry server node id to the m3u8 url as sessionId #9380
Conversation
- add entry server node id to the m3u8 url in case of MediaServerNode of type LiveCluster. - change getHttpUrl function in DeliveryProfileLive.php to receive entryServerNode instead of serverNode
| @@ -354,7 +355,10 @@ protected function getLivePackagerUrl($serverNode, $streamFormat = null) | |||
| $entryId = $this->getDynamicAttributes()->getEntryId(); | |||
| } | |||
|
|
|||
| $livePackagerUrl = "$livePackagerUrl/p/$partnerID/e/$entryId/" . $serverNode->getSegmentDurationUrlString($segmentDuration); | |||
| $livePackagerUrl = "$livePackagerUrl/p/$partnerID/e/$entryId/"; | |||
There was a problem hiding this comment.
Is there a chance in the future that we'll want both segmentDurationUrlString and sessionIdUrlString on the same url?
Isn't it always one or the other? seems like there should be one call and the relevant serverNode will handle it accordingly.
There was a problem hiding this comment.
You are correct - currently, one is for the old live and the other for the new - but, can it be that in the future we will have server that need them both? yes
we wonder if the wrap both as one function of get extraParams or something - then each server will call all his related function - but then we will need to change the wowza-serverNode code, I prefer the avoid that
There was a problem hiding this comment.
Better to leave the legacy code untouched I guess.
This change is