-
Notifications
You must be signed in to change notification settings - Fork 1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
youtube bridge: when specifying duration constraints, video descriptions get truncated. #4186
Comments
i dont think this is gonna get fixed. the obtained info is coming from json in the html source of https://www.youtube.com/channel/UCI3DTtB-a3fJPjKtQ5kYHfA/videos |
Does it come from a different place when no duration limit is specified? |
yes it comes from xml feed in that case. and it does not contain duration data. |
here is xml data from official feed: we could possibly scrape out more data from here but our naive feed parser does not handle the <feed xmlns:yt="http://www.youtube.com/xml/schemas/2015" xmlns:media="http://search.yahoo.com/mrss/" xmlns="http://www.w3.org/2005/Atom">
<link rel="self" href="http://www.youtube.com/feeds/videos.xml?channel_id=UCI3DTtB-a3fJPjKtQ5kYHfA"/>
<id>yt:channel:I3DTtB-a3fJPjKtQ5kYHfA</id>
<yt:channelId>I3DTtB-a3fJPjKtQ5kYHfA</yt:channelId>
<title>Games Done Quick</title>
<link rel="alternate" href="https://www.youtube.com/channel/UCI3DTtB-a3fJPjKtQ5kYHfA"/>
<author>
<name>Games Done Quick</name>
<uri>https://www.youtube.com/channel/UCI3DTtB-a3fJPjKtQ5kYHfA</uri>
</author>
<published>2014-07-19T15:21:36+00:00</published>
<entry>
<id>yt:video:IFWOG_KaV7w</id>
<yt:videoId>IFWOG_KaV7w</yt:videoId>
<yt:channelId>UCI3DTtB-a3fJPjKtQ5kYHfA</yt:channelId>
<title>Lenna's Inception by SnowballSMB in 57:56 - Flame Fatales 2024</title>
<link rel="alternate" href="https://www.youtube.com/watch?v=IFWOG_KaV7w"/>
<author>
<name>Games Done Quick</name>
<uri>https://www.youtube.com/channel/UCI3DTtB-a3fJPjKtQ5kYHfA</uri>
</author>
<published>2024-08-21T20:39:23+00:00</published>
<updated>2024-08-21T20:43:40+00:00</updated>
<media:group>
<media:title>Lenna's Inception by SnowballSMB in 57:56 - Flame Fatales 2024</media:title>
<media:content url="https://www.youtube.com/v/IFWOG_KaV7w?version=3" type="application/x-shockwave-flash" width="640" height="390"/>
<media:thumbnail url="https://i2.ytimg.com/vi/IFWOG_KaV7w/hqdefault.jpg" width="480" height="360"/>
<media:description>Runner introduction starts at 0:00
Run starts at 1:29
Commentary is provided by terpretor
savage_octagon is host
This speedrun was recorded during Flame Fatales 2024, a week long charity speedrun marathon raising money for Malala Fund. Flame Fatales 2024 is just one of the many charity marathons organized by Frame Fatales. For more information on Flame Fatales 2024, find us at: https://gamesdonequick.com/framefatales
Looking for live updates?
Follow us on Twitter: https://twitter.com/GamesDoneQuick
Like us on Facebook: http://www.facebook.com/GamesDoneQuick
Follow us on Instagram: https://instagram.com/gamesdonequick</media:description>
<media:community>
<media:starRating count="1" average="5.00" min="1" max="5"/>
<media:statistics views="34"/>
</media:community>
</media:group>
</entry>
|
Describe the bug
Video descriptions are truncated when duration_min or duration_max are not 0.
To Reproduce
Example feed with duration_min=2 so as to exclude shorts: https://rss-bridge.org/bridge01/?action=display&bridge=YoutubeBridge&context=By+channel+id&c=UCI3DTtB-a3fJPjKtQ5kYHfA&duration_min=2&format=Html
Example feed with duration_min=0 that shows full descriptions: https://rss-bridge.org/bridge01/?action=display&bridge=YoutubeBridge&context=By+channel+id&c=UCI3DTtB-a3fJPjKtQ5kYHfA&duration_min=0&format=Html
Removing the duration_min parameter altogether behaves the same as using a 0.
Expected behavior
I'd like the entire description to remain in the feed, independently of any duration constraints.
Screenshots
HTML output directly from the official instance, first with both duration parameters set to 0, then duration_min=2 like I want it
The text was updated successfully, but these errors were encountered: