IA appears to ignore BaseURL in representation in periods after the first

[matthuisman]

it doesnt switch the baseurl when going to the next period.
Always uses the baseurl from the first period (exchange219026anubr_219026_1600/hZrPcUpQ4/)

Log is me playing the 24 hour buffer live stream from the start.
It plays ok.
I then seek to future periods and start getting 404 due to it still trying to use the first periods baseurl

inputstream.adaptive v21.4.3
(doesnt seem to work on kodi 20 either)

manifest_1709440931.txt
kodi.log.txt

[CastagnaIT]

looks like for some reason this code dont work

inputstream.adaptive/src/Session.cpp

Lines 1101 to 1119 in fb6aead

	// Check if we leave our current period
	double chapterTime{0};
	auto pi = m_adaptiveTree->m_periods.cbegin();
	for (; pi != m_adaptiveTree->m_periods.cend(); pi++)
	{
	chapterTime += double((*pi)->GetDuration()) / (*pi)->GetTimescale();
	if (chapterTime > seekTime)
	break;
	}
	if (pi == m_adaptiveTree->m_periods.cend())
	--pi;
	chapterTime -= double((*pi)->GetDuration()) / (*pi)->GetTimescale();
	if ((*pi).get() != m_adaptiveTree->m_currentPeriod)
	{
	LOG::Log(LOGDEBUG, "SeekTime: seeking into new chapter: %d",

sound like a wrong chapter duration
you can see on the kodi GUI time bar the chapters points or not?
testing with master (omega) branch its the same?

[matthuisman]

different channel with 2x periods. Kodi 21 does show "chapter 1/2"
Seek near end then it freezes

manifest_1709624751.txt
kodi.log.txt

Just tested with latest master (997272c) and same issue

[CastagnaIT]

@matthuisman can you test with current last ISA release that i fixed mpd TSB and period start?

[CastagnaIT]

i have no dash multiperiod with so high timeshift time to test,
try apply this diff patch to see if improve the situation
the last period is missing the duration and i think this is cause of the problem
this change should add the duration to the last period (just a test maybe its possible do a more clean change)

diff --git a/src/parser/DASHTree.cpp b/src/parser/DASHTree.cpp
index e6537b4a..ab39e17f 100644
--- a/src/parser/DASHTree.cpp
+++ b/src/parser/DASHTree.cpp
@@ -222,10 +222,8 @@ bool adaptive::CDashTree::ParseManifest(const std::string& data)

   uint64_t totalDuration{0}; // Calculated duration, in ms
   uint64_t mpdTotalDuration = m_mediaPresDuration; // MPD total duration, in ms
-  if (mpdTotalDuration == 0)
-    mpdTotalDuration = m_timeShiftBufferDepth;

-  if (!IsLive())
+  //if (!IsLive())
   {
     for (auto itPeriod = m_periods.begin(); itPeriod != m_periods.end();)
     {
@@ -288,6 +286,9 @@ bool adaptive::CDashTree::ParseManifest(const std::string& data)
     }
   }

+  if (mpdTotalDuration == 0)
+    mpdTotalDuration = m_timeShiftBufferDepth;
+
   if (mpdTotalDuration > 0)
     m_totalTime = mpdTotalDuration;
   else