]> Untitled Git - lemmy.git/blobdiff - crates/apub/src/activities/community/announce.rs
When announcing incoming activities, keep extra fields (#2550)
[lemmy.git] / crates / apub / src / activities / community / announce.rs
index e5419680ef9fda18266e121d34b4cc5a90c008a9..f99a650fba592c4b5532ae232d7ec0b2c77cca0f 100644 (file)
@@ -1,10 +1,16 @@
 use crate::{
-  activities::{generate_activity_id, send_lemmy_activity, verify_is_public},
+  activities::{
+    generate_activity_id,
+    send_lemmy_activity,
+    verify_is_public,
+    verify_person_in_community,
+  },
   activity_lists::AnnouncableActivities,
   insert_activity,
   objects::community::ApubCommunity,
   protocol::{
-    activities::{community::announce::AnnounceActivity, CreateOrUpdateType},
+    activities::community::announce::{AnnounceActivity, RawAnnouncableActivities},
+    Id,
     IdOrNestedObject,
   },
   ActorType,
@@ -13,9 +19,59 @@ use activitypub_federation::{core::object_id::ObjectId, data::Data, traits::Acti
 use activitystreams_kinds::{activity::AnnounceType, public};
 use lemmy_utils::error::LemmyError;
 use lemmy_websocket::LemmyContext;
+use serde_json::Value;
 use tracing::debug;
 use url::Url;
 
+#[async_trait::async_trait(?Send)]
+impl ActivityHandler for RawAnnouncableActivities {
+  type DataType = LemmyContext;
+  type Error = LemmyError;
+
+  fn id(&self) -> &Url {
+    &self.id
+  }
+
+  fn actor(&self) -> &Url {
+    &self.actor
+  }
+
+  #[tracing::instrument(skip_all)]
+  async fn verify(
+    &self,
+    _data: &Data<Self::DataType>,
+    _request_counter: &mut i32,
+  ) -> Result<(), Self::Error> {
+    Ok(())
+  }
+
+  #[tracing::instrument(skip_all)]
+  async fn receive(
+    self,
+    data: &Data<Self::DataType>,
+    request_counter: &mut i32,
+  ) -> Result<(), Self::Error> {
+    let activity: AnnouncableActivities = self.clone().try_into()?;
+    // This is only for sending, not receiving so we reject it.
+    if let AnnouncableActivities::Page(_) = activity {
+      return Err(LemmyError::from_message("Cant receive page"));
+    }
+    let community = activity.get_community(data, &mut 0).await?;
+    let actor_id = ObjectId::new(activity.actor().clone());
+
+    // verify and receive activity
+    activity.verify(data, request_counter).await?;
+    activity.receive(data, request_counter).await?;
+
+    // send to community followers
+    if community.local {
+      verify_person_in_community(&actor_id, &community, data, &mut 0).await?;
+      AnnounceActivity::send(self, &community, data).await?;
+    }
+    Ok(())
+  }
+}
+
 #[async_trait::async_trait(?Send)]
 pub(crate) trait GetCommunity {
   async fn get_community(
@@ -27,7 +83,7 @@ pub(crate) trait GetCommunity {
 
 impl AnnounceActivity {
   pub(crate) fn new(
-    object: AnnouncableActivities,
+    object: RawAnnouncableActivities,
     community: &ApubCommunity,
     context: &LemmyContext,
   ) -> Result<AnnounceActivity, LemmyError> {
@@ -41,13 +97,12 @@ impl AnnounceActivity {
         &AnnounceType::Announce,
         &context.settings().get_protocol_and_hostname(),
       )?,
-      unparsed: Default::default(),
     })
   }
 
   #[tracing::instrument(skip_all)]
   pub async fn send(
-    object: AnnouncableActivities,
+    object: RawAnnouncableActivities,
     community: &ApubCommunity,
     context: &LemmyContext,
   ) -> Result<(), LemmyError> {
@@ -57,13 +112,18 @@ impl AnnounceActivity {
 
     // Pleroma and Mastodon can't handle activities like Announce/Create/Page. So for
     // compatibility, we also send Announce/Page so that they can follow Lemmy communities.
-    use AnnouncableActivities::*;
-    let object = match object {
-      CreateOrUpdatePost(c) if c.kind == CreateOrUpdateType::Create => Page(c.object),
-      _ => return Ok(()),
-    };
-    let announce_compat = AnnounceActivity::new(object, community, context)?;
-    send_lemmy_activity(context, announce_compat, community, inboxes, false).await?;
+    let object_parsed = object.try_into()?;
+    if let AnnouncableActivities::CreateOrUpdatePost(c) = object_parsed {
+      // Hack: need to convert Page into a format which can be sent as activity, which requires
+      //       adding actor field.
+      let announcable_page = RawAnnouncableActivities {
+        id: c.object.id.clone().into_inner(),
+        actor: c.actor.clone().into_inner(),
+        other: serde_json::to_value(c.object)?.as_object().unwrap().clone(),
+      };
+      let announce_compat = AnnounceActivity::new(announcable_page, community, context)?;
+      send_lemmy_activity(context, announce_compat, community, inboxes, false).await?;
+    }
     Ok(())
   }
 }
@@ -97,27 +157,53 @@ impl ActivityHandler for AnnounceActivity {
     context: &Data<LemmyContext>,
     request_counter: &mut i32,
   ) -> Result<(), LemmyError> {
-    let object = self.object.object(context, request_counter).await?;
+    let object: AnnouncableActivities = self
+      .object
+      .object(context, request_counter)
+      .await?
+      .try_into()?;
+    // This is only for sending, not receiving so we reject it.
+    if let AnnouncableActivities::Page(_) = object {
+      return Err(LemmyError::from_message("Cant receive page"));
+    }
+
     // we have to verify this here in order to avoid fetching the object twice over http
     object.verify(context, request_counter).await?;
 
-    // TODO: this can probably be implemented in a cleaner way
-    match object {
-      // Dont insert these into activities table, as they are not activities.
-      AnnouncableActivities::Page(_) => {}
-      _ => {
-        let object_value = serde_json::to_value(&object)?;
-        let insert =
-          insert_activity(object.id(), object_value, false, true, context.pool()).await?;
-        if !insert {
-          debug!(
-            "Received duplicate activity in announce {}",
-            object.id().to_string()
-          );
-          return Ok(());
-        }
-      }
+    let object_value = serde_json::to_value(&object)?;
+    let insert = insert_activity(object.id(), object_value, false, true, context.pool()).await?;
+    if !insert {
+      debug!(
+        "Received duplicate activity in announce {}",
+        object.id().to_string()
+      );
+      return Ok(());
     }
     object.receive(context, request_counter).await
   }
 }
+
+impl Id for RawAnnouncableActivities {
+  fn object_id(&self) -> &Url {
+    ActivityHandler::id(self)
+  }
+}
+
+impl TryFrom<RawAnnouncableActivities> for AnnouncableActivities {
+  type Error = serde_json::error::Error;
+
+  fn try_from(value: RawAnnouncableActivities) -> Result<Self, Self::Error> {
+    let mut map = value.other.clone();
+    map.insert("id".to_string(), Value::String(value.id.to_string()));
+    map.insert("actor".to_string(), Value::String(value.actor.to_string()));
+    serde_json::from_value(Value::Object(map))
+  }
+}
+
+impl TryFrom<AnnouncableActivities> for RawAnnouncableActivities {
+  type Error = serde_json::error::Error;
+
+  fn try_from(value: AnnouncableActivities) -> Result<Self, Self::Error> {
+    serde_json::from_value(serde_json::to_value(value)?)
+  }
+}