search

FreshRSS / Extensions

A repository containing all the official FreshRSS extensions
GNU Affero General Public License v3.0
322 stars 52 forks source link

[ReadingTime] YouTube Support #35

Open Wdavery opened 5 years ago

Wdavery commented 5 years ago

ReadingTime should either report the video length for YouTube video, or if that's too complicated, just ignore them altogether. "<1m" is not very helpful.

Alkarex commented 5 years ago

Maybe something for @kevinpapst or @PhieF ? :-)

kevinpapst commented 5 years ago

Hm, I don't think that really belongs to the Youtube plugin... as it neither communicates with Youtube and nor has any active frontend logic/JS. But it could be done with Javascript, there is a length_seconds parameter included in e.g. http://youtube.com/get_video_info?video_id=123456 There might be a better API, that was the first working version I could find.

Frenzie commented 5 years ago

There's the proper API, but since the deprecation of v2 it requires a dev key so it's not half as useful for generic use as it sounds at first glance: https://developers.google.com/youtube/v3/docs/videos

I'd just ignore it. :-)

Alkarex commented 5 years ago

The duration is not provided in the YouTube RSS feeds, so it would indeed require another request per video

lapineige commented 4 years ago

Then I could simply remove the reading time estimation for youtube videos... but I would have to do it for any kind of "video-only" content, and I don't see a good way to automate this.

(don't hesitate to ping me next time, I wasn't aware of this issue ^^)