Fix duplicate logging where both RequestHandled and CommandFinished events got fired during the same request

Author: onlime

CHANGELOG.md

@@ -1,10 +1,14 @@
 # CHANGELOG
 
-## [v1.2.x (Unreleased)](https://github.com/onlime/laravel-sql-reporter/compare/v1.2.2...main)
+## [v1.2.x (Unreleased)](https://github.com/onlime/laravel-sql-reporter/compare/v1.2.3...main)
 
-- Made datetime format of formatted header configurable and added TZ offset to default format `Y-m-d H:i:s P`.
 - ...
 
+## [v1.2.3 (2024-09-12)](https://github.com/onlime/laravel-sql-reporter/compare/v1.2.2...v1.2.3)
+
+- Made datetime format of formatted header configurable and added TZ offset to default format `Y-m-d H:i:s P`.
+- Fix | Prevent duplicate logging when a console command is called from a regular web request (e.g. programmatically executing an Artisan command with `Artisan::call()`). In web context, we're now only logging on `RequestHandled` event, while in console only on `CommandFinished` event.
+
 ## [v1.2.2 (2024-03-21)](https://github.com/onlime/laravel-sql-reporter/compare/v1.2.1...v1.2.2)
 
 - The `Writer` object now has a static `shouldReportSqlQuery()` method to define a custom callback for filtering queries included in the `QueryLogWritten` event. by @pascalbaljet in #4

src/Listeners/LogSqlQuery.php

@@ -4,9 +4,10 @@
 
 use Illuminate\Console\Events\CommandFinished;
 use Illuminate\Foundation\Http\Events\RequestHandled;
+use Illuminate\Support\Facades\App;
 use Onlime\LaravelSqlReporter\SqlLogger;
 
-class LogSqlQuery
+readonly class LogSqlQuery
 {
     /**
      * Create the event listener.
@@ -21,6 +22,12 @@ public function __construct(
      */
     public function handle(CommandFinished|RequestHandled $event): void
     {
+        // Prevent duplicate logging when running in request (RequestHandled event)
+        // and programmatically executing Artisan commands (CommandFinished event).
+        if ($event instanceof RequestHandled === App::runningInConsole()) {
+            return;
+        }
+
         $this->logger->log();
     }
 }