From: Dessalines Date: Mon, 3 Jul 2023 22:02:57 +0000 (-0400) Subject: 300 comment limit. (#3306) X-Git-Url: http://these/git/readmes/%7B%60%24%7BwebArchiveUrl%7D/save/static/%24%7Bargs.pageFn.jump%20n%7D?a=commitdiff_plain;h=f0e487f18a29afc0671eaf245801b80c442fdd1c;p=lemmy.git 300 comment limit. (#3306) * 300 comment limit. * Another fix. --- diff --git a/crates/api_common/src/utils.rs b/crates/api_common/src/utils.rs index f400cc9a..fd143ed9 100644 --- a/crates/api_common/src/utils.rs +++ b/crates/api_common/src/utils.rs @@ -670,7 +670,6 @@ pub async fn remove_user_data_in_community( .pool(pool) .creator_id(Some(banned_person_id)) .community_id(Some(community_id)) - .limit(Some(i64::MAX)) .build() .list() .await?; diff --git a/crates/db_views/src/comment_view.rs b/crates/db_views/src/comment_view.rs index 7c33158d..f3b1f83a 100644 --- a/crates/db_views/src/comment_view.rs +++ b/crates/db_views/src/comment_view.rs @@ -36,7 +36,7 @@ use lemmy_db_schema::{ post::Post, }, traits::JoinView, - utils::{fuzzy_search, get_conn, limit_and_offset_unlimited, DbPool}, + utils::{fuzzy_search, get_conn, limit_and_offset, DbPool}, CommentSortType, ListingType, }; @@ -340,9 +340,12 @@ impl<'a> CommentQuery<'a> { // This does not work for comment trees, and the limit should be manually set to a high number // // If a max depth is given, then you know its a tree fetch, and limits should be ignored - (i64::MAX, 0) + // TODO a kludge to prevent attacks. Limit comments to 300 for now. + // (i64::MAX, 0) + (300, 0) } else { - limit_and_offset_unlimited(self.page, self.limit) + // limit_and_offset_unlimited(self.page, self.limit) + limit_and_offset(self.page, self.limit)? }; query = match self.sort.unwrap_or(CommentSortType::Hot) {