fixed bug where custom events stopped recording

src/Helpers/FilterHelper.php

@@ -229,6 +229,13 @@ public static function recordOutboundRequestResponseBody($url, $status_code, $du
         }, false);
     }
 
+    public static function recordCustomEvent(string $name, array $meta = []): bool
+    {
+        return static::recordEvent(function () use ($name, $meta) {
+            return app(LaritorOverride::class)->recordCustomEvent($name, $meta);
+        });
+    }
+
     public static function whitelistedVendors(): array
     {
         return [];

src/Laritor.php

@@ -23,7 +23,7 @@
 
 class Laritor
 {
-    public const VERSION = '4.0.3';
+    public const VERSION = '4.1.0';
 
     /**
      * @var array
@@ -412,6 +412,7 @@ public function filterEvents()
                         RequestRecorder::$eventType => FilterHelper::recordRequest($event['request_instance'], $event['response_instance'], $event['response']['status_code'], $event['request']['duration']),
                         ScheduledTaskRecorder::$eventType => FilterHelper::recordCommandOrScheduledTask($event['task'], $event['status'], $event['duration'] ?? 0),
                         SchedulerRecorder::$eventType => FilterHelper::recordTaskScheduler(),
+                        self::CUSTOM_EVENT => FilterHelper::recordCustomEvent($event['name'], $event),
                         default => false
                     };
 

src/Override/DefaultOverride.php

@@ -283,4 +283,9 @@ public function whitelistedVendors(): array
     {
         return [];
     }
+
+    public function recordCustomEvent(string $name, array $meta = []): bool
+    {
+        return true;
+    }
 }

src/Override/LaritorOverride.php

@@ -146,5 +146,7 @@ public function recordOutboundRequestResponseHeaders($url, $status_code, $durati
 
     public function recordOutboundRequestResponseBody($url, $status_code, $duration): bool;
 
+    public function recordCustomEvent(string $name, array $meta = []): bool;
+
     public function whitelistedVendors(): array;
 }

stubs/ExceptionsOnlyDataFilter.stub

@@ -351,4 +351,9 @@ class LaritorDataFilter extends DefaultOverride
     {
          return parent::whitelistedVendors();
     }
+
+    public function recordCustomEvent(string $name, array $meta = []): bool
+    {
+        return Laritor::hasException();
+    }
 }

stubs/FullObservabilityDataFilter.stub

@@ -350,4 +350,9 @@ class LaritorDataFilter extends DefaultOverride
     {
         return parent::whitelistedVendors();
     }
+
+        public function recordCustomEvent(string $name, array $meta = []): bool
+        {
+            return true;
+        }
 }

stubs/IssuesOnlyDataFilter.stub

@@ -36,11 +36,7 @@ class LaritorDataFilter extends DefaultOverride
             $laritor->getCommandDuration() >= 300000;
     }
 
-    /**
-     * @param string $cacheKey
-     * @return bool
-     */
-    public function recordCacheHit($cacheKey): bool
+    public function hasIssue(): bool
     {
         return
             $this->hasFailedRequest() ||
@@ -49,6 +45,15 @@ class LaritorDataFilter extends DefaultOverride
             Laritor::hasException();
     }
 
+    /**
+     * @param string $cacheKey
+     * @return bool
+     */
+    public function recordCacheHit($cacheKey): bool
+    {
+        return $this->hasIssue();
+    }
+
     /**
      * @param \Throwable $exception
      * @return bool
@@ -67,10 +72,7 @@ class LaritorDataFilter extends DefaultOverride
     public function recordOutboundRequest($url, $status_code, $duration): bool
     {
         return
-            $this->hasFailedRequest() ||
-            $this->hasFailedJob() ||
-            $this->hasFailedCommand() ||
-            Laritor::hasException() ||
+            $this->hasIssue() ||
             $status_code >= 300 ||
             $duration >= 1000;
     }
@@ -84,10 +86,7 @@ class LaritorDataFilter extends DefaultOverride
     public function recordQuery($query, $duration, $path): bool
     {
         return
-            $this->hasFailedRequest() ||
-            $this->hasFailedJob() ||
-            $this->hasFailedCommand() ||
-            Laritor::hasException() ||
+            $this->hasIssue() ||
             $duration >= 200;
     }
 
@@ -102,9 +101,7 @@ class LaritorDataFilter extends DefaultOverride
     public function recordQueuedJob(string $connection, string $queue, string $job, string $status, $duration): bool
     {
         return
-            $this->hasFailedRequest() ||
-            $this->hasFailedCommand() ||
-            Laritor::hasException() ||
+            $this->hasIssue() ||
             strtolower($status) === 'failed' ||
             $duration >= 60000;
     }
@@ -152,11 +149,7 @@ class LaritorDataFilter extends DefaultOverride
      */
     public function recordMail($mailable, $to, $subject): bool
     {
-        return
-            $this->hasFailedRequest() ||
-            $this->hasFailedJob() ||
-            $this->hasFailedCommand() ||
-            Laritor::hasException();
+        return $this->hasIssue();
     }
 
     /**
@@ -166,11 +159,7 @@ class LaritorDataFilter extends DefaultOverride
      */
     public function recordNotification($notifiable, $notification): bool
     {
-        return
-            $this->hasFailedRequest() ||
-            $this->hasFailedJob() ||
-            $this->hasFailedCommand() ||
-            Laritor::hasException();
+        return $this->hasIssue();
     }
 
     /**
@@ -180,11 +169,7 @@ class LaritorDataFilter extends DefaultOverride
      */
     public function recordFeatureFlag($flag, $scope): bool
     {
-        return
-            $this->hasFailedRequest() ||
-            $this->hasFailedJob() ||
-            $this->hasFailedCommand() ||
-            Laritor::hasException();
+        return $this->hasIssue();
     }
 
     /**
@@ -195,12 +180,7 @@ class LaritorDataFilter extends DefaultOverride
      */
     public function recordLog($level, $message, array $context = []): bool
     {
-        return
-            $this->hasFailedRequest() ||
-            $this->hasFailedJob() ||
-            $this->hasFailedCommand() ||
-            Laritor::hasException() ||
-            !in_array(strtoupper($level), ['INFO','DEBUG']);
+        return $this->hasIssue() || !in_array(strtoupper($level), ['INFO','DEBUG']);
     }
 
     /**
@@ -418,4 +398,9 @@ class LaritorDataFilter extends DefaultOverride
     {
         return parent::whitelistedVendors();
     }
+
+    public function recordCustomEvent(string $name, array $meta = []): bool
+    {
+        return $this->hasIssue();
+    }
 }