]> Untitled Git - lemmy.git/commitdiff
Include type information with webfinger responses (fixes #2037) (#2746)
authorNutomic <me@nutomic.com>
Wed, 22 Feb 2023 02:25:26 +0000 (11:25 +0900)
committerGitHub <noreply@github.com>
Wed, 22 Feb 2023 02:25:26 +0000 (11:25 +0900)
So that it is easier to parse for other software

crates/routes/src/webfinger.rs
crates/utils/src/lib.rs

index 9056ea632be2d2f52ea5af06940d2f9d7ee764bb..4d744de939c2d99934a34cdebf552d90e19752ab 100644 (file)
@@ -7,6 +7,7 @@ use lemmy_db_schema::{
 };
 use lemmy_utils::{error::LemmyError, location_info, WebfingerLink, WebfingerResponse};
 use serde::Deserialize;
+use std::collections::HashMap;
 use url::Url;
 
 #[derive(Deserialize)]
@@ -53,8 +54,8 @@ async fn get_webfinger_response(
   // Mastodon seems to prioritize the last webfinger item in case of duplicates. Put
   // community last so that it gets prioritized. For Lemmy the order doesnt matter.
   let links = vec![
-    webfinger_link_for_actor(user_id),
-    webfinger_link_for_actor(community_id),
+    webfinger_link_for_actor(user_id, "Person"),
+    webfinger_link_for_actor(community_id, "Group"),
   ]
   .into_iter()
   .flatten()
@@ -68,18 +69,25 @@ async fn get_webfinger_response(
   Ok(HttpResponse::Ok().json(json))
 }
 
-fn webfinger_link_for_actor(url: Option<Url>) -> Vec<WebfingerLink> {
+fn webfinger_link_for_actor(url: Option<Url>, kind: &str) -> Vec<WebfingerLink> {
   if let Some(url) = url {
+    let mut properties = HashMap::new();
+    properties.insert(
+      "https://www.w3.org/ns/activitystreams#type".to_string(),
+      kind.to_string(),
+    );
     vec![
       WebfingerLink {
         rel: Some("http://webfinger.net/rel/profile-page".to_string()),
         kind: Some("text/html".to_string()),
         href: Some(url.clone()),
+        properties: Default::default(),
       },
       WebfingerLink {
         rel: Some("self".to_string()),
         kind: Some("application/activity+json".to_string()),
         href: Some(url),
+        properties,
       },
     ]
   } else {
index 48d16615294bdb7639ec4d2b46f5542e77f621cf..799d699454486dae6950610ffad4c0c4be571b8c 100644 (file)
@@ -15,7 +15,7 @@ pub mod utils;
 pub mod version;
 
 use serde::{Deserialize, Serialize};
-use std::{fmt, time::Duration};
+use std::{collections::HashMap, fmt, time::Duration};
 use url::Url;
 
 pub type ConnectionId = usize;
@@ -37,6 +37,7 @@ pub struct WebfingerLink {
   #[serde(rename = "type")]
   pub kind: Option<String>,
   pub href: Option<Url>,
+  pub properties: HashMap<String, String>,
 }
 
 #[derive(Serialize, Deserialize, Debug)]