activitypub transmogrifier: clean up accept/reject handling a bit

This commit is contained in:
William Pitcock 2018-05-25 09:38:07 +00:00
parent e80d91c64a
commit f35e6bf75b

View file

@ -171,7 +171,9 @@ def handle_incoming(
with %User{} = followed <- User.get_or_fetch_by_ap_id(actor),
{:ok, follow_activity} <- get_follow_activity(follow_object),
%User{local: true} = follower <- User.get_cached_by_ap_id(follow_activity["actor"]) do
User.follow(follower, followed)
if not User.following?(follower, followed) do
User.follow(follower, followed)
end
{:ok, data}
end
@ -182,10 +184,10 @@ def handle_incoming(
) do
with %User{} = followed <- User.get_or_fetch_by_ap_id(actor),
{:ok, follow_activity} <- get_follow_activity(follow_object),
%User{local: true} = follower <- User.get_cached_by_ap_id(follow_activity["actor"]),
{:ok, follow_activity} <- Utils.fetch_latest_follow(follower, followed),
{:ok, activity} <- ActivityPub.delete(follow_activity, false) do
{:ok, activity}
%User{local: true} = follower <- User.get_cached_by_ap_id(follow_activity["actor"]) do
User.unfollow(follower, followed)
{:ok, data}
end
end