temp: dont fetch remote profiles for remote membership updates
Signed-off-by: strawberry <strawberry@puppygock.gay>
This commit is contained in:
parent
e4789cfc23
commit
5c94caa3bb
1 changed files with 7 additions and 2 deletions
|
@ -2,7 +2,6 @@ use std::{collections::HashSet, sync::Arc};
|
||||||
|
|
||||||
pub use data::Data;
|
pub use data::Data;
|
||||||
use ruma::{
|
use ruma::{
|
||||||
api::federation,
|
|
||||||
events::{
|
events::{
|
||||||
direct::DirectEvent,
|
direct::DirectEvent,
|
||||||
ignored_user_list::IgnoredUserListEvent,
|
ignored_user_list::IgnoredUserListEvent,
|
||||||
|
@ -35,11 +34,16 @@ impl Service {
|
||||||
let membership = membership_event.membership;
|
let membership = membership_event.membership;
|
||||||
|
|
||||||
// Keep track what remote users exist by adding them as "deactivated" users
|
// Keep track what remote users exist by adding them as "deactivated" users
|
||||||
|
//
|
||||||
|
// TODO: use futures to update remote profiles without blocking the membership
|
||||||
|
// update
|
||||||
|
#[allow(clippy::collapsible_if)]
|
||||||
if user_id.server_name() != services().globals.server_name() {
|
if user_id.server_name() != services().globals.server_name() {
|
||||||
if !services().users.exists(user_id)? {
|
if !services().users.exists(user_id)? {
|
||||||
services().users.create(user_id, None)?;
|
services().users.create(user_id, None)?;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/*
|
||||||
// Try to update our local copy of the user if ours does not match
|
// Try to update our local copy of the user if ours does not match
|
||||||
if ((services().users.displayname(user_id)? != membership_event.displayname)
|
if ((services().users.displayname(user_id)? != membership_event.displayname)
|
||||||
|| (services().users.avatar_url(user_id)? != membership_event.avatar_url)
|
|| (services().users.avatar_url(user_id)? != membership_event.avatar_url)
|
||||||
|
@ -55,12 +59,13 @@ impl Service {
|
||||||
field: None, // we want the full user's profile to update locally too
|
field: None, // we want the full user's profile to update locally too
|
||||||
},
|
},
|
||||||
)
|
)
|
||||||
.await?;
|
.await;
|
||||||
|
|
||||||
services().users.set_displayname(user_id, response.displayname.clone()).await?;
|
services().users.set_displayname(user_id, response.displayname.clone()).await?;
|
||||||
services().users.set_avatar_url(user_id, response.avatar_url).await?;
|
services().users.set_avatar_url(user_id, response.avatar_url).await?;
|
||||||
services().users.set_blurhash(user_id, response.blurhash).await?;
|
services().users.set_blurhash(user_id, response.blurhash).await?;
|
||||||
};
|
};
|
||||||
|
*/
|
||||||
}
|
}
|
||||||
|
|
||||||
match &membership {
|
match &membership {
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue