fix: gate goblin logs before formatting (#3238)

### Description

As evaluated in the context of
#3237, in some
configurations, just formatting the `goblin` logs can be costly enough
to become a performance issue.

This PR moves the logging gate for `goblin` into `enabled()` so that we
discard its logs before we format them.

### Issues

Raised as part of the fixes for
getsentry/sentry#104738

### Legal Boilerplate
Look, I get it. The entity doing business as "Sentry" was incorporated
in the State of Delaware in 2015 as Functional Software, Inc. and is
gonna need some rights from me in order to utilize my contributions in
this here PR. So here's the deal: I retain all rights, title and
interest in and to my contributions, and by keeping this boilerplate
intact I confirm that Sentry can use, modify, copy, and redistribute my
contributions, under Sentry's choice of terms.

Author: supervacuus

src/utils/logging.rs

@@ -54,7 +54,7 @@ impl Logger {
 
 impl log::Log for Logger {
     fn enabled(&self, metadata: &log::Metadata) -> bool {
-        self.get_actual_level(metadata) <= max_level()
+        !should_skip_log(metadata) && self.get_actual_level(metadata) <= max_level()
     }
 
     fn log(&self, record: &log::Record) {
@@ -84,10 +84,6 @@ impl log::Log for Logger {
             .dim(),
         );
 
-        if should_skip_log(record) {
-            return;
-        }
-
         if let Some(pb) = get_progress_bar() {
             pb.println(msg);
         } else {
@@ -98,9 +94,9 @@ impl log::Log for Logger {
     fn flush(&self) {}
 }
 
-fn should_skip_log(record: &log::Record) -> bool {
-    let level = record.metadata().level();
-    let target = record.target();
+fn should_skip_log(metadata: &log::Metadata) -> bool {
+    let level = metadata.level();
+    let target = metadata.target();
 
     // We want to filter everything that is non-error from `goblin` crate,
     // as `symbolicator` is responsible for making sure all warnings are handled correctly.