search

glanceapp / glance

A self-hosted dashboard that puts all your feeds in one place
GNU Affero General Public License v3.0
8.48k stars 295 forks source link

Youtube feed `include-shorts` option also means no live videos appear on feed. #214

Open 0x00cl opened 2 months ago

0x00cl commented 2 months ago

I'm using glance 0.6.1 on linux.

This commit https://github.com/glanceapp/glance/commit/09eedc08c18c7a758b0fd65b7ed17b7ccadca1e8 adds the include-shorts option but I noticed that videos that are released in the Live tab don't appear when this option is enabled.

Lets take for example the Smosh channel. (ID: UCY30JRSgfhYXA6i6xX1erWg)

If the include-shorts option is false (default) then it will request this url https://www.youtube.com/feeds/videos.xml?playlist_id=UULFY30JRSgfhYXA6i6xX1erWg if its true then it requests https://www.youtube.com/feeds/videos.xml?channel_id=UCY30JRSgfhYXA6i6xX1erWg

With include-shorts false, the video that appears in the Live tab on youtube, like "Bit City LIVE Q+A!" doesn't appear in the list, while with the other option it does.

I feel this option should either be renamed or add a note so that it says that it will include live videos being filtered. I tried to look for ways to filter only shorts but the only thing I noticed is that shorts don't have a description, the XML is just a </media:description> but not sure if the XML is the same for "normal" videos with no description.

svilenmarkov commented 1 month ago

Thanks for reporting this.

After doing some digging I found out that you can replace the UULF with UULV like this and that will return a playlist with all videos from the Live tab. When #212 gets merged you should be able to just include that playlist ID and have live videos showing up again.