Skip to content
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

[RedditBridge] Initial commit #1213

Merged
merged 3 commits into from
Nov 1, 2019
Merged

[RedditBridge] Initial commit #1213

merged 3 commits into from
Nov 1, 2019

Conversation

Leomaradan
Copy link
Contributor

Reddit has native RSS feed, but Feedly can't handle them correctly.

This bridge will manage the reddit's feed, and has an option to "concat" multiple SubReddit feed into one

@somini
Copy link
Contributor

somini commented Sep 21, 2019

https://validator.w3.org/feed/check.cgi?url=https%3A%2F%2Fwww.reddit.com%2Fr%2Fselfhosted%2F.rss

It seems the feed is not exactly valid, but our Atom parser is very liberal.

RSS Bridge output is in fact a valid Atom feed, so this looks good to me.

bridges/RedditBridge.php Outdated Show resolved Hide resolved
Copy link
Contributor

@logmanoriginal logmanoriginal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your contribution, this looks very interesting 👍

Find below a few comments regarding the implementation.

bridges/RedditBridge.php Outdated Show resolved Hide resolved
bridges/RedditBridge.php Outdated Show resolved Hide resolved
bridges/RedditBridge.php Outdated Show resolved Hide resolved
@logmanoriginal
Copy link
Contributor

Works for me. Thanks again 👍

@logmanoriginal logmanoriginal merged commit 46d5895 into RSS-Bridge:master Nov 1, 2019
@Leomaradan
Copy link
Contributor Author

I find some issue if the SubReddit doesn't exist. That cause the whole feed to be rejected. I need to add a new test for non-existant subreddit (eg. https://www.reddit.com/r/react/.rss return a 403 code)

infominer33 pushed a commit to web-work-tools/rss-bridge that referenced this pull request Apr 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants