From 39909dc17c25d146433faedb2f1a6466f1178e75 Mon Sep 17 00:00:00 2001 From: Martin Linzmayer Date: Fri, 31 Jan 2025 12:04:28 +0100 Subject: [PATCH 1/2] fix(relay): set size limit for user report attachments --- relay-server/src/utils/sizes.rs | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/relay-server/src/utils/sizes.rs b/relay-server/src/utils/sizes.rs index 96ec13e3de..36dbcb59ea 100644 --- a/relay-server/src/utils/sizes.rs +++ b/relay-server/src/utils/sizes.rs @@ -43,7 +43,7 @@ pub fn check_envelope_size_limits(config: &Config, envelope: &Envelope) -> Resul event_size += item.len(); NO_LIMIT } - ItemType::Attachment | ItemType::UnrealReport => { + ItemType::Attachment | ItemType::UnrealReport | ItemType::UserReport => { attachments_size += item.len(); config.max_attachment_size() } @@ -59,7 +59,6 @@ pub fn check_envelope_size_limits(config: &Config, envelope: &Envelope) -> Resul } ItemType::Profile => config.max_profile_size(), ItemType::CheckIn => config.max_check_in_size(), - ItemType::UserReport => NO_LIMIT, ItemType::Statsd => config.max_statsd_size(), ItemType::MetricBuckets => config.max_metric_buckets_size(), ItemType::Log => config.max_log_size(), From 85659c92cccf19c9369292436d63d515d2b46f80 Mon Sep 17 00:00:00 2001 From: Martin Linzmayer Date: Fri, 31 Jan 2025 12:17:39 +0100 Subject: [PATCH 2/2] changelog --- CHANGELOG.md | 1 + 1 file changed, 1 insertion(+) diff --git a/CHANGELOG.md b/CHANGELOG.md index f75ddd9f03..fceea4a075 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -13,6 +13,7 @@ **Bug Fixes**: - Fix a bug where parsing large unsigned integers would fail incorrectly. ([#4472](https://github.com/getsentry/relay/pull/4472)) +- Set size limit for UserReport attachments so it gets properly reported as too large. ([#4482](https://github.com/getsentry/relay/pull/4482)) **Internal**: