fix(ExceptionSerializer): encode arguments before filtering the trace

This will avoid running into a Nesting level too deep error as the encodeArg calls will limit potential recursive calls on the arguments to a nesting level of 5 Signed-off-by: Julius Härtl <[email protected]>
nextcloud · Jan 18, 2023 · a2073f4 · a2073f4
1 parent ae03129
commit a2073f4
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/lib/private/Log/ExceptionSerializer.php b/lib/private/Log/ExceptionSerializer.php
@@ -228,13 +228,13 @@ private function removeValuesFromArgs($args, $values) {
 	}
 
 	private function encodeTrace($trace) {
-		$filteredTrace = $this->filterTrace($trace);
-		return array_map(function (array $line) {
+		$trace = array_map(function (array $line) {
 			if (isset($line['args'])) {
 				$line['args'] = array_map([$this, 'encodeArg'], $line['args']);
 			}
 			return $line;
-		}, $filteredTrace);
+		}, $trace);
+		return $this->filterTrace($trace);
 	}
 
 	private function encodeArg($arg, $nestingLevel = 5) {