fix: Ensure namespaces are only matched on metrics

getsentry · Mar 27, 2024 · 094a864 · 094a864
1 parent cc18860
commit 094a864
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/relay-quotas/src/rate_limit.rs b/relay-quotas/src/rate_limit.rs
@@ -2,6 +2,7 @@ use std::fmt;
 use std::str::FromStr;
 use std::time::{Duration, Instant};
 
+use relay_base_schema::data_category::DataCategory;
 use relay_base_schema::metrics::MetricNamespace;
 use relay_base_schema::project::{ProjectId, ProjectKey};
 use smallvec::SmallVec;
@@ -182,7 +183,7 @@ impl RateLimit {
 
     /// Returns `true` if the rate limit namespace matches the namespace of the item.
     fn matches_namespace(&self, scoping: ItemScoping<'_>) -> bool {
-        if self.namespace.is_empty() {
+        if self.namespace.is_empty() || scoping.category != DataCategory::MetricBucket {
             true
         } else if let Some(namespace) = scoping.namespace {
             self.namespace.contains(&namespace)