Merge pull request #10 from bethinkpl/IT-3172-limit-max-spans

IT-3172 | Limit maximum number of spans per one transaction
bethinkpl · Nov 19, 2020 · 93a0551 · 93a0551
2 parents 2f31645 + ae3b58c
commit 93a0551
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 0 deletions.
diff --git a/config/elastic-apm.php b/config/elastic-apm.php
@@ -51,6 +51,9 @@
     ],
 
     'spans' => [
+        // Max number of child items displayed when viewing trace details.
+        'maxTraceItems' => env('APM_MAXTRACEITEMS', 1000),
+
         // Depth of backtraces
         'backtraceDepth'=> env('APM_BACKTRACEDEPTH', 25),
 

diff --git a/src/Apm/SpanCollection.php b/src/Apm/SpanCollection.php
@@ -11,5 +11,12 @@
  */
 class SpanCollection extends Collection
 {
+    public function push($value)
+    {
+        if ($this->count() >= config('elastic-apm.spans.maxTraceItems')) {
+            return;
+        }
 
+        parent::push($value);
+    }
 }