Fix some federation issue
#573
Samengevoegd
Plume_migration_agent
heeft 6 commits samengevoegd van fix/minor-federation-issue
naar master
5 jaren geleden
Laden…
Verwijs in nieuw issue
Er is nog geen inhoud.
Verwijder branch 'fix/minor-federation-issue'
Het verwijderen van een branch is permanent. Het KAN NIET ongedaan gemaakt worden. Wil je toch doorgaan?
The issue in #567 will be a bit tougher than initially expected. Pterotype seems to use Organization instead of plain User for federation, so there is the question how do we manage that? How do we treat that? As users I expect, but it will require some changes to do so, or as something else?
A post must have an associated blog, but it technically can have no author. So I would say Organizations should be treated as blogs.
You can probably create a type that implements
AsActor
and the differentAsObject
, but that actually delegates the work toBlog
? I don't know.atm a post can't really have no authors : in multiple places assumption is made that there is at least one author, and we can use
posts.get_authors()?[0]
safely. There will be things to change if we introduce the possibility of a post without authors. Also as one can't follow a blog yet, this would mean a basic feature is lacking. The more I think about it, the more problematic it getCodecov Report
OK, so then, what about inserting both a blog and an user with the same username when encountering an
Organization
? It is a bit hackish, but it should work.Also, we should maybe add a "default" personnal blog for users at some, as I think it is what most people expect (but it may make the whole code a lot more complex too)?
This may be useful to cunsult for reference:
https://github.com/w3c/activitypub/issues/260
There may (or may not) be some similarity to this Pleroma issue:
https://git.pleroma.social/pleroma/pleroma/issues/364
@fdb-hiroshima @BaptisteGelez According to the creator of Pterotype,
Organization
is the entire blog, not a specific user in the current design of the plugin.I think this may help to shed some light on the issue at hand:
He plans to redesign the federation approach in the future though, and test it against Plume himself then.
this has been stall for too long. What has been fixed has been fixed, the rest will be at some point in the future
What you have done looks good 👍 If you could just cancel the changes to the translation files (or fix the conflicts with master, but the former is probably easier), we could merge this PR.
Wait I just realized: do we fetch blogs with the
group:
prefix in Webfinger now?no, we don't 😬
This time it looks good, thanks!
Reviewers
4f7c20fc26
.Stap 1:
Vanuit het project, check een branch uit en test de veranderingen.Stap 2:
Voeg de wijzigingen samen en update ze op Forgejo.