sentry: Simplify init() function

`sentry::init()` already expects an `Option<Dsn>` so there is no reason for us to only call it if the env var exists. This reduces the nesting and simplfies the big `.map()` implementation.

Author: Turbo87

src/sentry.rs

@@ -1,5 +1,4 @@
 use sentry::{ClientInitGuard, ClientOptions, IntoDsn};
-use std::borrow::Cow;
 
 /// Initializes the Sentry SDK from the environment variables.
 ///
@@ -9,28 +8,26 @@ use std::borrow::Cow;
 ///
 /// `HEROKU_SLUG_COMMIT`, if present, will be used as the `release` property
 /// on all events.
-#[must_use]
-pub fn init() -> Option<ClientInitGuard> {
-    dotenv::var("SENTRY_DSN_API")
+pub fn init() -> ClientInitGuard {
+    let dsn = dotenv::var("SENTRY_DSN_API")
         .ok()
         .into_dsn()
-        .expect("SENTRY_DSN_API is not a valid Sentry DSN value")
-        .map(|dsn| {
-            let environment = Some(
-                dotenv::var("SENTRY_ENV_API")
-                    .map(Cow::Owned)
-                    .expect("SENTRY_ENV_API must be set when using SENTRY_DSN_API"),
-            );
+        .expect("SENTRY_DSN_API is not a valid Sentry DSN value");
 
-            let release = dotenv::var("HEROKU_SLUG_COMMIT").ok().map(Into::into);
+    let environment = dsn.as_ref().map(|_| {
+        dotenv::var("SENTRY_ENV_API")
+            .expect("SENTRY_ENV_API must be set when using SENTRY_DSN_API")
+            .into()
+    });
 
-            let opts = ClientOptions {
-                dsn: Some(dsn),
-                environment,
-                release,
-                ..Default::default()
-            };
+    let release = dotenv::var("HEROKU_SLUG_COMMIT").ok().map(Into::into);
 
-            sentry::init(opts)
-        })
+    let opts = ClientOptions {
+        dsn,
+        environment,
+        release,
+        ..Default::default()
+    };
+
+    sentry::init(opts)
 }