Revert "misc typing cleanup"

This reverts commit 4dd809fdc4.

src/service/rooms/read_receipt/mod.rs

@@ -59,7 +59,7 @@ impl Service {
 			.expect("room flush failed");
 		// update appservices
 		let edu = EphemeralData::Receipt(event);
-		_ = self
+		let _ = self
 			.services
 			.sending
 			.send_edu_appservice_room(

src/service/rooms/typing/mod.rs

@@ -146,34 +146,32 @@ impl Service {
 			}
 		};
 
-		if removable.is_empty() {
-			return Ok(());
-		}
+		if !removable.is_empty() {
+			let typing = &mut self.typing.write().await;
+			let room = typing.entry(room_id.to_owned()).or_default();
+			for user in &removable {
+				debug_info!("typing timeout {user:?} in {room_id:?}");
+				room.remove(user);
+			}
 
-		let typing = &mut self.typing.write().await;
-		let room = typing.entry(room_id.to_owned()).or_default();
-		for user in &removable {
-			debug_info!("typing timeout {user:?} in {room_id:?}");
-			room.remove(user);
-		}
+			// update clients
+			self.last_typing_update
+				.write()
+				.await
+				.insert(room_id.to_owned(), self.services.globals.next_count()?);
 
-		// update clients
-		self.last_typing_update
-			.write()
-			.await
-			.insert(room_id.to_owned(), self.services.globals.next_count()?);
+			if self.typing_update_sender.send(room_id.to_owned()).is_err() {
+				trace!("receiver found what it was looking for and is no longer interested");
+			}
 
-		if self.typing_update_sender.send(room_id.to_owned()).is_err() {
-			trace!("receiver found what it was looking for and is no longer interested");
-		}
+			// update appservices
+			self.appservice_send(room_id).await?;
 
-		// update appservices
-		self.appservice_send(room_id).await?;
-
-		// update federation
-		for user in &removable {
-			if self.services.globals.user_is_local(user) {
-				self.federation_send(room_id, user, false).await?;
+			// update federation
+			for user in &removable {
+				if self.services.globals.user_is_local(user) {
+					self.federation_send(room_id, user, false).await?;
+				}
 			}
 		}
 
@@ -192,17 +190,17 @@ impl Service {
 			.unwrap_or(0))
 	}
 
-	/// Returns a new typing EDU's content.
-	pub async fn typings_content(&self, room_id: &RoomId) -> TypingEventContent {
+	/// Returns a new typing EDU.
+	pub async fn typings_content(&self, room_id: &RoomId) -> Result<TypingEventContent> {
 		let room_typing_indicators = self.typing.read().await.get(room_id).cloned();
 
 		let Some(typing_indicators) = room_typing_indicators else {
-			return TypingEventContent { user_ids: Vec::new() };
+			return Ok(TypingEventContent { user_ids: Vec::new() });
 		};
 
 		let user_ids: Vec<_> = typing_indicators.into_keys().collect();
 
-		TypingEventContent { user_ids }
+		Ok(TypingEventContent { user_ids })
 	}
 
 	/// Returns a new typing EDU.
@@ -210,13 +208,13 @@ impl Service {
 		&self,
 		room_id: &RoomId,
 		sender_user: &UserId,
-	) -> SyncEphemeralRoomEvent<TypingEventContent> {
+	) -> Result<SyncEphemeralRoomEvent<TypingEventContent>> {
 		let room_typing_indicators = self.typing.read().await.get(room_id).cloned();
 
 		let Some(typing_indicators) = room_typing_indicators else {
-			return SyncEphemeralRoomEvent {
+			return Ok(SyncEphemeralRoomEvent {
 				content: TypingEventContent { user_ids: Vec::new() },
-			};
+			});
 		};
 
 		let user_ids: Vec<_> = typing_indicators
@@ -233,7 +231,7 @@ impl Service {
 			.collect()
 			.await;
 
-		SyncEphemeralRoomEvent { content: TypingEventContent { user_ids } }
+		Ok(SyncEphemeralRoomEvent { content: TypingEventContent { user_ids } })
 	}
 
 	async fn federation_send(
@@ -256,12 +254,14 @@ impl Service {
 		self.services
 			.sending
 			.send_edu_room(room_id, serde_json::to_vec(&edu).expect("Serialized Edu::Typing"))
-			.await
+			.await?;
+
+		Ok(())
 	}
 
 	async fn appservice_send(&self, room_id: &RoomId) -> Result<()> {
 		let edu = EphemeralData::Typing(EphemeralRoomEvent {
-			content: self.typings_content(room_id).await,
+			content: self.typings_content(room_id).await?,
 			room_id: room_id.into(),
 		});
 
@@ -271,6 +271,8 @@ impl Service {
 				room_id,
 				serde_json::to_vec(&edu).expect("Serialized EphemeralData::Typing"),
 			)
-			.await
+			.await?;
+
+		Ok(())
 	}
 }

src/service/sending/mod.rs

@@ -190,8 +190,8 @@ impl Service {
 	}
 
 	#[tracing::instrument(skip(self, serialized), level = "debug")]
-	pub fn send_edu_appservice(&self, appservice_id: &str, serialized: Vec<u8>) -> Result {
-		let dest = Destination::Appservice(appservice_id.to_owned());
+	pub fn send_edu_appservice(&self, appservice_id: String, serialized: Vec<u8>) -> Result {
+		let dest = Destination::Appservice(appservice_id);
 		let event = SendingEvent::Edu(serialized);
 		let _cork = self.db.db.cork();
 		let keys = self.db.queue_requests(once((&event, &dest)));
@@ -224,7 +224,7 @@ impl Service {
 					.appservice_in_room(room_id, appservice)
 					.await
 			{
-				self.send_edu_appservice(&appservice.registration.id, serialized.clone())?;
+				self.send_edu_appservice(appservice.registration.id.clone(), serialized.clone())?;
 			}
 		}
 		Ok(())