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

Breaks on Mastodon RSS feed #79

Closed
captn3m0 opened this issue Jun 5, 2023 · 3 comments · Fixed by #82
Closed

Breaks on Mastodon RSS feed #79

captn3m0 opened this issue Jun 5, 2023 · 3 comments · Fixed by #82

Comments

@captn3m0
Copy link

captn3m0 commented Jun 5, 2023

I've been trying to use this RSS feed, from an account I run: https://tatooine.club/@dykindia.rss

Relevant config is straight forward:

    "provider": "rss",
            "config": {
                "rss_path": "https://tatooine.club/@dykindia.rss",
                "limit": 1
            }
        }

However, the result is very confusing, since it seems to be getting a noscript tag that's not present in the actual RSS feed.

image

I've been running goosepaper with docker, so haven't gotten around to debugging it yet.

@j6k4m8
Copy link
Owner

j6k4m8 commented Jun 5, 2023

Hey @captn3m0, thanks so much for the bug report! This looks to me like we're definitely "trying too hard" to get RSS content from the web; I think the answer here is for us to have a separate Mastodon provider so that we can parse the (relatively consistent) ActivityPub responses from Mastodon servers. Thoughts?

@captn3m0
Copy link
Author

captn3m0 commented Jun 7, 2023

I don't understand the "trying too hard" part - given a properly formatted RSS feed, it should parse the feed right? Why doesn't that work?

It might be easier to use the Mastodon API if we go down this route, instead of the ActivityPub, which might require too much work.

@j6k4m8
Copy link
Owner

j6k4m8 commented Aug 7, 2023

hey @captn3m0 thanks for the help tracking this down — added a Mastodon StoryProvider in #82 just for this! Will be on PyPI in a few min.

@j6k4m8 j6k4m8 closed this as completed in #82 Aug 7, 2023
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 a pull request may close this issue.

2 participants