dawansv wrote: 
> For me the latest version seems to be working so far, including the
> piece that is failing for Mordac.
> 
> But I am curious how do you know that in Mordac's case HTML::Entities
> are not being used, or are you just assuming that based on the fact it
> fails later? When I look at the code, the http content is printed to the
> log on line 365 before the clean up occurs, no? So is what we are seeing
> in the log not just the raw content before HTML:Entities is involved?
> 
> But it is true that since the log mentions "adaptive_fmts" on line 3121
> of the log, then it implies that the search for adaptiveFormats failed
> and it shifted to the next item to try which is "adaptive_fmts". On a
> side note, that code could be strengthened a bit 
> because it actually fails as well but there is no check in that case,
> hence why the log prints "adaptive_fmts" info as blank.
> 
That's just an assumption for the lack of anything better. Yes, the log
is purposely made before the cleanup so that I can look at it if needed.
But as it seems that exactly as you say, the search of adaptiveFormats
fails which would mean that $content returned by decode_entities has an
issue of some sort

> 
> To me to be certain the clean up failed it seems it would help to output
> $content to the log just before the check for adaptiveFormats occurs on
> line 390 to see what it looks like at that point. And you are right that
> we might see it hasn“t been cleaned up which would explain the fail. 
> 
> > 
Code:
--------------------
  >   > my ($streams) = $content =~ /\"adaptiveFormats\":(\[.*?\])/;
--------------------
> > 
> 
> Edit: And I confirm that https://www.youtube.com/watch?v=94Wk-YgSEDI
> also returns the "old" format for me, with the backslashes, so in my
> case it seems that "decode_entities" is doing a fine job cleaning it
> up.
I'll wait a bit for feedback from others as it works for me 100% now,
while nothing was working when I decided to make the change (yes, I've
been lazy, it was working up to recently and I did not look at it before
I was impacted myself - not too many people complained and I was pretty
busy



LMS 7.9  on Pi 3B+ & Odroid-C2 - *SqueezeAMP!*, 5xRadio, 3xBoom, 4xDuet,
1xTouch, 1 SB3. Sonos PLAY:3, PLAY:5, Marantz NR1603, Foobar2000,
ShairPortW, JRiver 21, 2xChromecast Audio, Chromecast v1 and v2,
Squeezelite on Pi,  Yamaha WX-010, AppleTV 4, Airport Express, GGMM E5,
Riva 1 & 3
------------------------------------------------------------------------
philippe_44's Profile: http://forums.slimdevices.com/member.php?userid=17261
View this thread: http://forums.slimdevices.com/showthread.php?t=105840

_______________________________________________
plugins mailing list
[email protected]
http://lists.slimdevices.com/mailman/listinfo/plugins

Reply via email to