use crate::{
check_apub_id_valid_with_strictness,
fetch_local_site_data,
- generate_outbox_url,
objects::{instance::fetch_instance_actor_for_object, read_from_string_or_source_opt},
protocol::{
objects::{
ImageObject,
Source,
},
- ActorType,
};
use activitypub_federation::{
- core::object_id::ObjectId,
- traits::{Actor, ApubObject},
- utils::verify_domains_match,
+ config::Data,
+ protocol::verification::verify_domains_match,
+ traits::{Actor, Object},
};
use chrono::NaiveDateTime;
-use lemmy_api_common::utils::{blocking, local_site_opt_to_slur_regex};
+use lemmy_api_common::{
+ context::LemmyContext,
+ utils::{generate_outbox_url, local_site_opt_to_slur_regex},
+};
use lemmy_db_schema::{
- source::{
- instance::Instance,
- person::{Person as DbPerson, PersonInsertForm, PersonUpdateForm},
- },
+ source::person::{Person as DbPerson, PersonInsertForm, PersonUpdateForm},
traits::{ApubActor, Crud},
utils::naive_now,
};
use lemmy_utils::{
error::LemmyError,
- utils::{check_slurs, check_slurs_opt, convert_datetime, markdown_to_html},
+ utils::{
+ markdown::markdown_to_html,
+ slurs::{check_slurs, check_slurs_opt},
+ time::convert_datetime,
+ },
};
-use lemmy_websocket::LemmyContext;
use std::ops::Deref;
use url::Url;
#[derive(Clone, Debug, PartialEq, Eq)]
-pub struct ApubPerson(DbPerson);
+pub struct ApubPerson(pub(crate) DbPerson);
impl Deref for ApubPerson {
type Target = DbPerson;
}
}
-#[async_trait::async_trait(?Send)]
-impl ApubObject for ApubPerson {
+#[async_trait::async_trait]
+impl Object for ApubPerson {
type DataType = LemmyContext;
- type ApubType = Person;
- type DbType = DbPerson;
+ type Kind = Person;
type Error = LemmyError;
fn last_refreshed_at(&self) -> Option<NaiveDateTime> {
}
#[tracing::instrument(skip_all)]
- async fn read_from_apub_id(
+ async fn read_from_id(
object_id: Url,
- context: &LemmyContext,
+ context: &Data<Self::DataType>,
) -> Result<Option<Self>, LemmyError> {
Ok(
- blocking(context.pool(), move |conn| {
- DbPerson::read_from_apub_id(conn, &object_id.into())
- })
- .await??
- .map(Into::into),
+ DbPerson::read_from_apub_id(context.pool(), &object_id.into())
+ .await?
+ .map(Into::into),
)
}
#[tracing::instrument(skip_all)]
- async fn delete(self, context: &LemmyContext) -> Result<(), LemmyError> {
- blocking(context.pool(), move |conn| {
- let form = PersonUpdateForm::builder().deleted(Some(true)).build();
- DbPerson::update(conn, self.id, &form)
- })
- .await??;
+ async fn delete(self, context: &Data<Self::DataType>) -> Result<(), LemmyError> {
+ let form = PersonUpdateForm::builder().deleted(Some(true)).build();
+ DbPerson::update(context.pool(), self.id, &form).await?;
Ok(())
}
#[tracing::instrument(skip_all)]
- async fn into_apub(self, _pool: &LemmyContext) -> Result<Person, LemmyError> {
+ async fn into_json(self, _context: &Data<Self::DataType>) -> Result<Person, LemmyError> {
let kind = if self.bot_account {
UserTypes::Service
} else {
let person = Person {
kind,
- id: ObjectId::new(self.actor_id.clone()),
+ id: self.actor_id.clone().into(),
preferred_username: self.name.clone(),
name: self.display_name.clone(),
summary: self.bio.as_ref().map(|b| markdown_to_html(b)),
endpoints: self.shared_inbox_url.clone().map(|s| Endpoints {
shared_inbox: s.into(),
}),
- public_key: self.get_public_key(),
+ public_key: self.public_key(),
updated: self.updated.map(convert_datetime),
inbox: self.inbox_url.clone().into(),
};
async fn verify(
person: &Person,
expected_domain: &Url,
- context: &LemmyContext,
- _request_counter: &mut i32,
+ context: &Data<Self::DataType>,
) -> Result<(), LemmyError> {
- let local_site_data = blocking(context.pool(), fetch_local_site_data).await??;
+ let local_site_data = fetch_local_site_data(context.pool()).await?;
let slur_regex = &local_site_opt_to_slur_regex(&local_site_data.local_site);
check_slurs(&person.preferred_username, slur_regex)?;
}
#[tracing::instrument(skip_all)]
- async fn from_apub(
+ async fn from_json(
person: Person,
- context: &LemmyContext,
- request_counter: &mut i32,
+ context: &Data<Self::DataType>,
) -> Result<ApubPerson, LemmyError> {
- let apub_id = person.id.inner().to_owned();
- let instance = blocking(context.pool(), move |conn| {
- Instance::create_from_actor_id(conn, &apub_id)
- })
- .await??;
+ let instance_id = fetch_instance_actor_for_object(&person.id, context).await?;
+
+ // Some Mastodon users have `name: ""` (empty string), need to convert that to `None`
+ // https://github.com/mastodon/mastodon/issues/25233
+ let display_name = person.name.filter(|n| !n.is_empty());
let person_form = PersonInsertForm {
name: person.preferred_username,
- display_name: person.name,
+ display_name,
banned: None,
ban_expires: None,
- deleted: None,
+ deleted: Some(false),
avatar: person.icon.map(|i| i.url.into()),
banner: person.image.map(|i| i.url.into()),
published: person.published.map(|u| u.naive_local()),
inbox_url: Some(person.inbox.into()),
shared_inbox_url: person.endpoints.map(|e| e.shared_inbox.into()),
matrix_user_id: person.matrix_user_id,
- instance_id: instance.id,
+ instance_id,
};
- let person = blocking(context.pool(), move |conn| {
- DbPerson::create(conn, &person_form)
- })
- .await??;
-
- let actor_id = person.actor_id.clone().into();
- fetch_instance_actor_for_object(actor_id, context, request_counter).await;
+ let person = DbPerson::upsert(context.pool(), &person_form).await?;
Ok(person.into())
}
}
-impl ActorType for ApubPerson {
- fn actor_id(&self) -> Url {
- self.actor_id.to_owned().into()
+impl Actor for ApubPerson {
+ fn id(&self) -> Url {
+ self.actor_id.inner().clone()
}
- fn private_key(&self) -> Option<String> {
- self.private_key.to_owned()
+ fn public_key_pem(&self) -> &str {
+ &self.public_key
}
-}
-impl Actor for ApubPerson {
- fn public_key(&self) -> &str {
- &self.public_key
+ fn private_key_pem(&self) -> Option<String> {
+ self.private_key.clone()
}
fn inbox(&self) -> Url {
}
fn shared_inbox(&self) -> Option<Url> {
- self.shared_inbox_url.clone().map(|s| s.into())
+ self.shared_inbox_url.clone().map(Into::into)
}
}
},
protocol::{objects::instance::Instance, tests::file_to_json_object},
};
+ use activitypub_federation::fetch::object_id::ObjectId;
use lemmy_db_schema::{source::site::Site, traits::Crud};
use serial_test::serial;
- pub(crate) async fn parse_lemmy_person(context: &LemmyContext) -> (ApubPerson, ApubSite) {
+ pub(crate) async fn parse_lemmy_person(context: &Data<LemmyContext>) -> (ApubPerson, ApubSite) {
let site = parse_lemmy_instance(context).await;
let json = file_to_json_object("assets/lemmy/objects/person.json").unwrap();
let url = Url::parse("https://enterprise.lemmy.ml/u/picard").unwrap();
- let mut request_counter = 0;
- ApubPerson::verify(&json, &url, context, &mut request_counter)
- .await
- .unwrap();
- let person = ApubPerson::from_apub(json, context, &mut request_counter)
- .await
- .unwrap();
- assert_eq!(request_counter, 0);
+ ApubPerson::verify(&json, &url, context).await.unwrap();
+ let person = ApubPerson::from_json(json, context).await.unwrap();
+ assert_eq!(context.request_count(), 0);
(person, site)
}
- #[actix_rt::test]
+ #[tokio::test]
#[serial]
async fn test_parse_lemmy_person() {
- let context = init_context();
+ let context = init_context().await;
let (person, site) = parse_lemmy_person(&context).await;
assert_eq!(person.display_name, Some("Jean-Luc Picard".to_string()));
assert!(!person.local);
assert_eq!(person.bio.as_ref().unwrap().len(), 39);
- cleanup((person, site), &context);
+ cleanup((person, site), &context).await;
}
- #[actix_rt::test]
+ #[tokio::test]
#[serial]
async fn test_parse_pleroma_person() {
- let context = init_context();
+ let context = init_context().await;
// create and parse a fake pleroma instance actor, to avoid network request during test
let mut json: Instance = file_to_json_object("assets/lemmy/objects/instance.json").unwrap();
- let id = Url::parse("https://queer.hacktivis.me/").unwrap();
- json.id = ObjectId::new(id);
- let mut request_counter = 0;
- let site = ApubSite::from_apub(json, &context, &mut request_counter)
- .await
- .unwrap();
+ json.id = ObjectId::parse("https://queer.hacktivis.me/").unwrap();
+ let url = Url::parse("https://queer.hacktivis.me/users/lanodan").unwrap();
+ ApubSite::verify(&json, &url, &context).await.unwrap();
+ let site = ApubSite::from_json(json, &context).await.unwrap();
let json = file_to_json_object("assets/pleroma/objects/person.json").unwrap();
- let url = Url::parse("https://queer.hacktivis.me/users/lanodan").unwrap();
- let mut request_counter = 0;
- ApubPerson::verify(&json, &url, &context, &mut request_counter)
- .await
- .unwrap();
- let person = ApubPerson::from_apub(json, &context, &mut request_counter)
- .await
- .unwrap();
+ ApubPerson::verify(&json, &url, &context).await.unwrap();
+ let person = ApubPerson::from_json(json, &context).await.unwrap();
assert_eq!(person.actor_id, url.into());
assert_eq!(person.name, "lanodan");
assert!(!person.local);
- assert_eq!(request_counter, 0);
+ assert_eq!(context.request_count(), 0);
assert_eq!(person.bio.as_ref().unwrap().len(), 873);
- cleanup((person, site), &context);
+ cleanup((person, site), &context).await;
}
- fn cleanup(data: (ApubPerson, ApubSite), context: &LemmyContext) {
- let conn = &mut context.pool().get().unwrap();
- DbPerson::delete(conn, data.0.id).unwrap();
- Site::delete(conn, data.1.id).unwrap();
+ async fn cleanup(data: (ApubPerson, ApubSite), context: &LemmyContext) {
+ DbPerson::delete(context.pool(), data.0.id).await.unwrap();
+ Site::delete(context.pool(), data.1.id).await.unwrap();
}
}