FediMovies/fedimovies
1
1
Fork 0
Code Issues 3 Pull requests Projects Releases Packages Activity

Allow Tag objects without name property

Browse source
This commit is contained in:
silverpill 2022-04-27 09:53:52 +00:00
parent 7a47c28034
commit b384dcd4d2
2 changed files with 20 additions and 11 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
src/activitypub/activity.rs
View file

@ -36,7 +36,7 @@ fn default_tag_type() -> String { HASHTAG.to_string() }
#[derive(Deserialize, Serialize)]
#[serde(rename_all = "camelCase")]
pub struct Tag {
pub name: String,
pub name: Option<String>,
#[serde(rename = "type", default = "default_tag_type")]
pub tag_type: String,
@ -204,11 +204,11 @@ pub fn create_note(
};
let mut tags = vec![];
for profile in &post.mentions {
let tag_name = format!("@{}", profile.actor_address(instance_host));
let actor_id = profile.actor_id(instance_url);
primary_audience.push(actor_id);
let actor_id = profile.actor_id(instance_url);
primary_audience.push(actor_id.clone());
let tag = Tag {
name: format!("@{}", profile.actor_address(instance_host)),
name: Some(tag_name),
tag_type: MENTION.to_string(),
href: Some(actor_id),
};
@ -217,7 +217,7 @@ pub fn create_note(
for tag_name in &post.tags {
let tag_page_url = get_tag_page_url(instance_url, tag_name);
let tag = Tag {
name: format!("#{}", tag_name),
name: Some(format!("#{}", tag_name)),
tag_type: HASHTAG.to_string(),
href: Some(tag_page_url),
};
@ -594,7 +594,7 @@ mod tests {
);
let tags = note.tag;
assert_eq!(tags.len(), 1);
assert_eq!(tags[0].name, format!("@{}", parent_author_acct));
assert_eq!(tags[0].name.as_deref().unwrap(), format!("@{}", parent_author_acct));
assert_eq!(tags[0].href.as_ref().unwrap(), parent_author_actor_id);
assert_eq!(note.to, vec![AP_PUBLIC, parent_author_actor_id]);
}

19
src/activitypub/receiver.rs
View file

@ -303,9 +303,11 @@ pub async fn import_post(
if let Some(list) = object.tag {
for tag in list {
if tag.tag_type == HASHTAG {
// Ignore invalid tags
if let Ok(tag_name) = normalize_tag(&tag.name) {
tags.push(tag_name);
if let Some(tag_name) = tag.name {
// Ignore invalid tags
if let Ok(tag_name) = normalize_tag(&tag_name) {
tags.push(tag_name);
};
};
} else if tag.tag_type == MENTION {
// Try to find profile by actor ID.
@ -339,9 +341,16 @@ pub async fn import_post(
};
};
// Try to find profile by actor address
let tag_name = match tag.name {
Some(name) => name,
None => {
log::warn!("failed to parse mention");
continue;
},
};
if let Ok(actor_address) = mention_to_address(
&instance.host(),
&tag.name,
&tag_name,
) {
let profile = match get_profile_by_acct(
db_client,
@ -372,7 +381,7 @@ pub async fn import_post(
mentions.push(profile.id);
};
} else {
log::warn!("failed to parse mention {}", tag.name);
log::warn!("failed to parse mention {}", tag_name);
};
};
};