summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorEllie Huxtable <ellie@elliehuxtable.com>2024-03-05 09:42:10 +0000
committerGitHub <noreply@github.com>2024-03-05 09:42:10 +0000
commit799dd1b0a96b8a80d7a08e05b212a86144f46195 (patch)
treedb05dd8fc3005ead97afef59e99dcb6922514eaf
parentb1155873ddcb33777474fd9f15f370b54cdd44f5 (diff)
fix(sync): record size limiter (#1827)
Settings the record size limit to 0 had issues. Inverting the condition works better.
-rw-r--r--atuin-server/src/handlers/v0/record.rs6
1 files changed, 3 insertions, 3 deletions
diff --git a/atuin-server/src/handlers/v0/record.rs b/atuin-server/src/handlers/v0/record.rs
index e1b5d29c..321c34c2 100644
--- a/atuin-server/src/handlers/v0/record.rs
+++ b/atuin-server/src/handlers/v0/record.rs
@@ -27,11 +27,11 @@ pub async fn post<DB: Database>(
counter!("atuin_record_uploaded", records.len() as u64);
- let too_big = records
+ let keep = records
.iter()
- .any(|r| r.data.data.len() >= settings.max_record_size || settings.max_record_size == 0);
+ .all(|r| r.data.data.len() <= settings.max_record_size || settings.max_record_size == 0);
- if too_big {
+ if !keep {
counter!("atuin_record_too_large", 1);
return Err(