search

cn-tools / cntools_FreshRssExtensions

This repository contains my unofficial FreshRSS extensions
MIT License
47 stars 9 forks source link

Use official youtube feed URL #6

Closed ljoonal closed 9 months ago

ljoonal commented 1 year ago

I don't see why this uses a third party feed instead of just using the feeds that YouTube provides itself. This PR would fix that.

kosukesando commented 9 months ago

I think CntYTRssHookBeforeInsertFeedfunction itself is unnecessary, no?

cn-tools commented 9 months ago

@ljoonal thx 4 your PR and i'm so sorry for the really late reply.

The reason for using 'scriptbarrel.com' is that it provides 50 entries instead of 15. Yes, this normaly has an effect when adding the feed only.

However, so that there is no longer any third-party provider involved here, i merge it.

kosukesando commented 9 months ago

I see. FYI, scriptbarrel wasn't able to handle Japanese characters correctly (perhaps it doesn't handle UTF-8 well) and was returning a corrupt feed, so I think this PR is good in that regard as well.