-
Notifications
You must be signed in to change notification settings - Fork 68
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
Improve lemmy compatibility #752
Conversation
That's great! You can test it with voyager.lemmy.ml which has the latest changes deployed. |
@Nutomic can you approve my request? |
Ive approved it. |
@Nutomic I messed up my password and it seems to not send password reset mails 🫣 |
It should be able to send emails, maybe it went to spam? Otherwise just register another account. |
made a second account |
@Nutomic OK, seems to work, at least following, but posts seems to not be federated properly?!? Here is an example JSON: https://pfefferle.org/hello-lemmy-part2/?activitypub |
I was able to fetch it without any problems. Maybe Lemmy couldnt parse the activity you sent? Can you show the json for that? |
All test posts here: https://pfefferle.org/ You can add |
I will try to debug the Lemmy response later today! |
FYI: The post will be federated by the User and Announced by the Group. But the Announce Object is only the Link to the Post, is that supported by Lemmy or does it need the full object as the Announce object? |
@Nutomic the request is:
The response is:
|
Tried it with object instead of URI, with the same result: Error 400
|
Right Lemmy only supports Announce with embedded object for now, no link. Ive thrown the last activity you sent into our tests and it passes just fine (after replacing Looking at voyager.lemmy.ml, it appears that you sent an activity with the same id |
OK, I have a working version now! 🎉 @Nutomic any chance that Lemmy will support URI objects soon? I am not sure if it is a good idea to change the current implementation, because I fear to break something else... 🫣 |
Unfortunately I wont be able to implement that anytime soon. |
@Nutomic ok, then I will see how other platforms handle Announce objects, to get this merged. |
@Nutomic It seems that I receive a 400 error on the Can you maybe help with debugging? Here is an example of an {
"@context": [
"https:\/\/www.w3.org\/ns\/activitystreams"
],
"id": "https:\/\/pfefferle.org\/@pfefferle.org#follow-lemmy.ml\/u\/pfefferle-1717694756",
"type": "Accept",
"to": "https:\/\/lemmy.ml\/u\/pfefferle",
"object": {
"id": "https:\/\/lemmy.ml\/activities\/follow\/2fdfb3f2-9f7e-44bf-bf26-c0d2cb371ebc",
"type": "Follow",
"object": "https:\/\/pfefferle.org\/@pfefferle.org",
"actor": "https:\/\/lemmy.ml\/u\/pfefferle"
},
"actor": "https:\/\/pfefferle.org\/@pfefferle.org"
} |
Improve the compatibility with lemmy! Thanks @Nutomic for the feedback https://socialhub.activitypub.rocks/t/discourse-wordpress-integration/4162/13?u=pfefferle