From: Felix Ableitner <me@nutomic.com>
Date: Fri, 7 Aug 2020 13:19:08 +0000 (+0200)
Subject: Merge branch 'main' into federation-authorisation
X-Git-Url: http://these/git/%7BtotpUrl%7D?a=commitdiff_plain;h=0cc49e6ca985f06792712c00d9be50b0d7c7eacd;p=lemmy.git

Merge branch 'main' into federation-authorisation
---

0cc49e6ca985f06792712c00d9be50b0d7c7eacd
diff --cc server/src/apub/community.rs
index 3773b8fb,549d9349..8b522b44
--- a/server/src/apub/community.rs
+++ b/server/src/apub/community.rs
@@@ -341,30 -334,50 +341,56 @@@ impl FromApub for CommunityForm 
        .unwrap();
  
      let creator = get_or_fetch_and_upsert_user(creator_uri, client, pool).await?;
 +    let name = group
 +      .inner
 +      .name()
 +      .unwrap()
 +      .as_one()
 +      .unwrap()
 +      .as_xsd_string()
 +      .unwrap()
 +      .to_string();
 +    let title = group.inner.preferred_username().unwrap().to_string();
 +    // TODO: should be parsed as html and tags like <script> removed (or use markdown source)
 +    //       -> same for post.content etc
 +    let description = group
 +      .inner
 +      .content()
 +      .map(|s| s.as_single_xsd_string().unwrap().into());
 +    check_slurs(&name)?;
 +    check_slurs(&title)?;
 +    check_slurs_opt(&description)?;
  
+     let icon = match group.icon() {
+       Some(any_image) => Some(
+         Image::from_any_base(any_image.as_one().unwrap().clone())
+           .unwrap()
+           .unwrap()
+           .url()
+           .unwrap()
+           .as_single_xsd_any_uri()
+           .map(|u| u.to_string()),
+       ),
+       None => None,
+     };
+ 
+     let banner = match group.image() {
+       Some(any_image) => Some(
+         Image::from_any_base(any_image.as_one().unwrap().clone())
+           .unwrap()
+           .unwrap()
+           .url()
+           .unwrap()
+           .as_single_xsd_any_uri()
+           .map(|u| u.to_string()),
+       ),
+       None => None,
+     };
+ 
      Ok(CommunityForm {
 -      name: group
 -        .inner
 -        .name()
 -        .unwrap()
 -        .as_one()
 -        .unwrap()
 -        .as_xsd_string()
 -        .unwrap()
 -        .into(),
 -      title: group.inner.preferred_username().unwrap().to_string(),
 -      // TODO: should be parsed as html and tags like <script> removed (or use markdown source)
 -      //       -> same for post.content etc
 -      description: group
 -        .inner
 -        .content()
 -        .map(|s| s.as_single_xsd_string().unwrap().into()),
 +      name,
 +      title,
 +      description,
        category_id: group.ext_one.category.identifier.parse::<i32>()?,
        creator_id: creator.id,
        removed: None,
diff --cc server/src/apub/user.rs
index eb042da9,c20335fe..58338ab4
--- a/server/src/apub/user.rs
+++ b/server/src/apub/user.rs
@@@ -207,20 -211,30 +213,35 @@@ impl ActorType for User_ 
  impl FromApub for UserForm {
    type ApubType = PersonExt;
    /// Parse an ActivityPub person received from another instance into a Lemmy user.
 -  async fn from_apub(person: &PersonExt, _: &Client, _: &DbPool) -> Result<Self, LemmyError> {
 +  async fn from_apub(
 +    person: &PersonExt,
 +    _: &Client,
 +    _: &DbPool,
 +    expected_domain: Option<Url>,
 +  ) -> Result<Self, LemmyError> {
      let avatar = match person.icon() {
-       Some(any_image) => Image::from_any_base(any_image.as_one().unwrap().clone())
-         .unwrap()
-         .unwrap()
-         .url()
-         .unwrap()
-         .as_single_xsd_any_uri()
-         .map(|u| u.to_string()),
+       Some(any_image) => Some(
+         Image::from_any_base(any_image.as_one().unwrap().clone())
+           .unwrap()
+           .unwrap()
+           .url()
+           .unwrap()
+           .as_single_xsd_any_uri()
+           .map(|u| u.to_string()),
+       ),
+       None => None,
+     };
+ 
+     let banner = match person.image() {
+       Some(any_image) => Some(
+         Image::from_any_base(any_image.as_one().unwrap().clone())
+           .unwrap()
+           .unwrap()
+           .url()
+           .unwrap()
+           .as_single_xsd_any_uri()
+           .map(|u| u.to_string()),
+       ),
        None => None,
      };