Merge pull request #863 from bakaphp/hotfix-queue

bakaphp · May 18, 2023 · a1aa2c5 · a1aa2c5
2 parents c5dad83 + 0fdd45f
commit a1aa2c5
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 6 deletions.
diff --git a/src/Cli/Tasks/QueueTask.php b/src/Cli/Tasks/QueueTask.php
@@ -161,7 +161,7 @@ public function jobsAction(?string $queueName = null)
                     return;
                 }
 
-                go(function () use ($job, $msg, $sentryClient) {
+                go(function () use ($job, $msg) {
                     //instance notification and pass the entity
                     try {
                         $this->reconnectDb();
@@ -179,10 +179,6 @@ public function jobsAction(?string $queueName = null)
                                 $e->getTraceAsString(),
                             ]
                         );
-
-                        if ($sentryClient) {
-                            $sentryClient->flush();
-                        }
                     }
                 });
             } catch (Throwable $e) {

diff --git a/src/Providers/LoggerProvider.php b/src/Providers/LoggerProvider.php
@@ -14,6 +14,7 @@
 use Sentry\ClientBuilder;
 use Sentry\Monolog\Handler as SentryHandler;
 use Sentry\State\Hub;
+use function Baka\isCLI;
 
 class LoggerProvider implements ServiceProviderInterface
 {
@@ -43,7 +44,7 @@ function () use ($container) {
                 $handler->setFormatter($formatter);
 
                 //only run logs in production
-                if ((bool) envValue('SENTRY_PROJECT', 0)) {
+                if ((bool) envValue('SENTRY_PROJECT', 0) && !isCLI()) {
                     //sentry logger
                     $client = ClientBuilder::create([
                         'dsn' => 'https://' . getenv('SENTRY_PROJECT_SECRET') . '@sentry.io/' . getenv('SENTRY_PROJECT_ID')