default to shared room history visibility if invalid (per spec)

Signed-off-by: strawberry <strawberry@puppygock.gay>

src/api/client_server/directory.rs

@@ -254,8 +254,8 @@ pub(crate) async fn get_public_rooms_filtered_helper(
 					.map_or(Ok(None), |s| {
 						serde_json::from_str(s.content.get())
 							.map(|c: RoomTopicEventContent| Some(c.topic))
-							.map_err(|_| {
-								error!("Invalid room topic event in database for room {}", room_id);
+							.map_err(|e| {
+								error!("Invalid room topic event in database for room {room_id}: {e}");
 								Error::bad_database("Invalid room topic event in database.")
 							})
 					})
@@ -271,11 +271,10 @@ pub(crate) async fn get_public_rooms_filtered_helper(
 							})
                             .map_err(|e| {
                                 error!(
-                                    "Invalid room history visibility event in database for room {}: {e}",
-                                    &room_id
+                                    "Invalid room history visibility event in database for room {room_id}, assuming is \"shared\": {e}",
                                 );
                                 Error::bad_database("Invalid room history visibility event in database.")
-                            })})?,
+                            })}).unwrap_or(false),
 				guest_can_join: services()
 					.rooms
 					.state_accessor

src/service/rooms/spaces/mod.rs

@@ -832,15 +832,22 @@ fn guest_can_join(room_id: &RoomId) -> Result<bool, Error> {
 
 /// Checks if guests are able to view room content without joining
 fn world_readable(room_id: &RoomId) -> Result<bool, Error> {
-	services()
+	Ok(services()
 		.rooms
 		.state_accessor
 		.room_state_get(room_id, &StateEventType::RoomHistoryVisibility, "")?
 		.map_or(Ok(false), |s| {
 			serde_json::from_str(s.content.get())
 				.map(|c: RoomHistoryVisibilityEventContent| c.history_visibility == HistoryVisibility::WorldReadable)
-				.map_err(|_| Error::bad_database("Invalid room history visibility event in database."))
+				.map_err(|e| {
+					error!(
+						"Invalid room history visibility event in database for room {room_id}, assuming is \
+						 \"shared\": {e} "
+					);
+					Error::bad_database("Invalid room history visibility event in database.")
+				})
 		})
+		.unwrap_or(false))
 }
 
 /// Returns the join rule for a given room

src/service/rooms/state_accessor/mod.rs

@@ -107,8 +107,15 @@ pub fn server_can_see_event(&self, origin: &ServerName, room_id: &RoomId, event_
 			.map_or(Ok(HistoryVisibility::Shared), |s| {
 				serde_json::from_str(s.content.get())
 					.map(|c: RoomHistoryVisibilityEventContent| c.history_visibility)
-					.map_err(|_| Error::bad_database("Invalid history visibility event in database."))
-			})?;
+					.map_err(|e| {
+						error!(
+							"Invalid history visibility event in database for room {room_id}, assuming is \"shared\": \
+							 {e}"
+						);
+						Error::bad_database("Invalid history visibility event in database.")
+					})
+			})
+			.unwrap_or(HistoryVisibility::Shared);
 
 		let mut current_server_members = services()
 			.rooms
@@ -165,8 +172,15 @@ pub fn user_can_see_event(&self, user_id: &UserId, room_id: &RoomId, event_id: &
 			.map_or(Ok(HistoryVisibility::Shared), |s| {
 				serde_json::from_str(s.content.get())
 					.map(|c: RoomHistoryVisibilityEventContent| c.history_visibility)
-					.map_err(|_| Error::bad_database("Invalid history visibility event in database."))
-			})?;
+					.map_err(|e| {
+						error!(
+							"Invalid history visibility event in database for room {room_id}, assuming is \"shared\": \
+							 {e}"
+						);
+						Error::bad_database("Invalid history visibility event in database.")
+					})
+			})
+			.unwrap_or(HistoryVisibility::Shared);
 
 		let visibility = match history_visibility {
 			HistoryVisibility::WorldReadable => true,
@@ -205,10 +219,14 @@ pub fn user_can_see_state_events(&self, user_id: &UserId, room_id: &RoomId) -> R
 				serde_json::from_str(s.content.get())
 					.map(|c: RoomHistoryVisibilityEventContent| c.history_visibility)
 					.map_err(|e| {
-						error!("Invalid history visibility event in database for room {}: {e}", &room_id);
+						error!(
+							"Invalid history visibility event in database for room {room_id}, assuming is \"shared\": \
+							 {e}"
+						);
 						Error::bad_database("Invalid history visibility event in database.")
 					})
-			})?;
+			})
+			.unwrap_or(HistoryVisibility::Shared);
 
 		Ok(currently_member || history_visibility == HistoryVisibility::WorldReadable)
 	}