fix: only capture 500 errors in sentry

mozilla-services · Nov 21, 2018 · b2935f5 · b2935f5
1 parent b12b07f
commit b2935f5
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/src/db/error.rs b/src/db/error.rs
@@ -65,7 +65,13 @@ impl From<Context<DbErrorKind>> for DbError {
             _ => StatusCode::INTERNAL_SERVER_ERROR,
         };
 
-        Self { inner, status }
+        let error = Self { inner, status };
+
+        if status == StatusCode::INTERNAL_SERVER_ERROR {
+            sentry::integrations::failure::capture_fail(&error);
+        }
+
+        error
     }
 }
 

diff --git a/src/error.rs b/src/error.rs
@@ -132,9 +132,7 @@ macro_rules! failure_boilerplate {
 
         impl From<$kind> for $error {
             fn from(kind: $kind) -> Self {
-                let error = Context::new(kind).into();
-                sentry::integrations::failure::capture_fail(&error);
-                error
+                Context::new(kind).into()
             }
         }
     };