-
Notifications
You must be signed in to change notification settings - Fork 1.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
[Mastodon] Use ActivityPub outbox for Mastodon (et al.) feed #2756
Conversation
I mean I could rename it to ActivityPub bridge if the maintainer so pleases
Pull request artifacts
|
Thanks for this fast work! I will be able to review this in more detail in about a week or two. For now, I noticed a few things:
|
I will wait till there is a consensus on renaming it. But wouldn't inheriting it add two duplicating feeds? |
I refactored a bit. Don't use |
@dvikan To be honest I'd also prefer for it to be a separate bridge. The current client should be stripped of its options. So if we're on the same page on this then I'll rename the files a bit. |
In that case, What about changing the |
I suggest to follow yamanq's advice. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work!
I have added a change to support instances with Secure Mode. This requires some setup by the host (which requires another PR to the docs), which can be summarized as:
For examples, try See also https://blog.joinmastodon.org/2018/06/how-to-implement-a-basic-activitypub-server/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, please add those instructions to securing a token into the Bridge Specific section of the docs. Also, mention that this bridge can be configured in the DESCRIPTION.
To satisfy the linter, you can just create a newline without closing the string:
const DESCRIPTION = 'Returns recent statuses. Supports Mastodon, Pleroma and Misskey, among others.
Some Mastodon instances use Secure Mode which ...';
Thanks for all your work on this! |
[Mastodon] Use ActivityPub outbox for Mastodon (et al.) feed (RSS-Bridge#2756)
Closes #2754.
In theory this supports all ActivityPub-compatible software1. This may warrant renaming the bridge (not included in PR).
Example: https://rssbridge.bus-hit.me/?action=display&bridge=Mastodon&canusername=%40austin%40ieji.de&format=Html
Footnotes
I have tested it on Pleroma and Misskey instances. AFAIK it supports certain Pixelfed instances, but the official one has ActivityPub outbox disabled. ↩