Merge pull request #5 from worksome/feature/enable

feat: support disabling Horizon Telemetry
worksome · Feb 1, 2023 · 789721a · 789721a
2 parents 08561b2 + fbfe37b
commit 789721a
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/composer.json b/composer.json
@@ -20,7 +20,7 @@
         "illuminate/contracts": "^9.46",
         "laravel/horizon": "^5.10",
         "open-telemetry/api": "^0.0.17",
-        "worksome/laravel-telemetry": "^0.2.0"
+        "worksome/laravel-telemetry": "^0.2.4"
     },
     "require-dev": {
         "nunomaduro/collision": "^6.4",

diff --git a/src/HorizonTelemetryServiceProvider.php b/src/HorizonTelemetryServiceProvider.php
@@ -32,6 +32,10 @@ public function boot(): void
             /** @var Repository $config */
             $config = $this->app->make(Repository::class);
 
+            if (! $config->get('telemetry.enabled', true)) {
+                return null;
+            }
+
             if ($config->get(self::CONFIG_PREFIX . MeterName::FailedJobs->value, true)) {
                 $dispatcher->listen(JobFailed::class, FailedJobsListener::class);
             }
@@ -50,6 +54,10 @@ public function boot(): void
             /** @var Repository $config */
             $config = $this->app->make(Repository::class);
 
+            if (! $config->get('telemetry.enabled', true)) {
+                return null;
+            }
+
             if ($currentMasterSupervisorsSchedule = $config->get(
                 self::CONFIG_PREFIX . MeterName::CurrentMasterSupervisors->value
             )) {