Merge pull request #170 from fupa/DV-15754-disable-verbose-job-logging

make verbose job logging configurable but disable it by default
brefphp · Feb 12, 2025 · e644c76 · e644c76
2 parents 50afe06 + b25c7f1
commit e644c76
Show file tree

Hide file tree

Showing 2 changed files with 27 additions and 7 deletions.
diff --git a/config/bref.php b/config/bref.php
@@ -30,4 +30,14 @@
 
     'request_context' => false,
 
+    /*
+    |--------------------------------------------------------------------------
+    | Jobs Logging
+    |--------------------------------------------------------------------------
+    |
+    | Here you can disable detailed logging of every job execution.
+    |
+    */
+
+    'log_jobs' => true,
 ];
diff --git a/src/BrefServiceProvider.php b/src/BrefServiceProvider.php
@@ -73,6 +73,23 @@ public function boot(Dispatcher $dispatcher, LogManager $logManager, FailedJobPr
             ], 'bref-config');
         }
 
+        if (config('bref.log_jobs', true)) {
+            $this->enableDetailedJobLogging($dispatcher, $logManager, $queueFailer);
+        }
+
+        if (file_exists('/proc/1/fd/1')) {
+            $dispatcher->listen(
+                ScheduledTaskStarting::class,
+                fn(ScheduledTaskStarting $task) => $task->task->appendOutputTo('/proc/1/fd/1'),
+            );
+        }
+    }
+
+    private function enableDetailedJobLogging(
+        Dispatcher $dispatcher,
+        LogManager $logManager,
+        FailedJobProviderInterface $queueFailer
+    ): void {
         $dispatcher->listen(
             fn (JobProcessing $event) => $logManager->info(
                 "Processing job {$event->job->getJobId()}",
@@ -102,13 +119,6 @@ public function boot(Dispatcher $dispatcher, LogManager $logManager, FailedJobPr
                 $event->exception
             )
         );
-
-        if (file_exists('/proc/1/fd/1')) {
-            $dispatcher->listen(
-                ScheduledTaskStarting::class,
-                fn(ScheduledTaskStarting $task) => $task->task->appendOutputTo('/proc/1/fd/1'),
-            );
-        }
     }
 
     /**