Merge pull request #91 from mozilla-services/pb/sentry-fixes

Two tiny tweaks for Sentry

Cargo.toml

@@ -7,6 +7,10 @@ authors = [
   "Phil Jenvey <[email protected]>",
 ]
 
+[profile.release]
+# Enables line numbers in Sentry
+debug = 1
+
 [dependencies]
 actix = "0.7"
 actix-web = "0.7.7"

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
     }
 }
 

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()
             }
         }
     };