Avoid occurring chained warnings by parse_str()

rollbar · Apr 12, 2022 · 86b4483 · 86b4483
1 parent bb0e839
commit 86b4483
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/src/DataBuilder.php b/src/DataBuilder.php
@@ -630,7 +630,8 @@ protected function getRequest()
             $postData = array();
             $body = $request->getBody();
             if ($body !== null) {
-                parse_str($body, $postData);
+                // PHP reports warning if parse_str() detects more than max_input_vars items.
+                @parse_str($body, $postData);
             }
             $request->setPost($postData);
         }

diff --git a/src/Scrubber.php b/src/Scrubber.php
@@ -80,7 +80,8 @@ public function internalScrub(&$data, $fields, $replacement, $path)
                     $data
                 );
             } else {
-                parse_str($data, $parsedData);
+                // PHP reports warning if parse_str() detects more than max_input_vars items.
+                @parse_str($data, $parsedData);
                 if (http_build_query($parsedData) === $data) {
                     $data = $this->scrubQueryString($data, $fields);
                 }
@@ -131,7 +132,8 @@ protected function scrubArray(&$arr, $fields, $replacement = '********', $path =
 
     protected function scrubQueryString($query, $fields, $replacement = 'xxxxxxxx')
     {
-        parse_str($query, $parsed);
+        // PHP reports warning if parse_str() detects more than max_input_vars items.
+        @parse_str($query, $parsed);
         $scrubbed = $this->internalScrub($parsed, $fields, $replacement, '');
         return http_build_query($scrubbed);
     }