From 52839f63a5e07be7a5c3f4e11633a6ef06609fdb Mon Sep 17 00:00:00 2001 From: Arthur Schiwon Date: Thu, 16 Jun 2022 13:50:59 +0200 Subject: [PATCH] make placeholder a const for reuse Signed-off-by: Arthur Schiwon --- lib/private/Log/ExceptionSerializer.php | 4 +++- tests/lib/Log/ExceptionSerializerTest.php | 2 +- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/lib/private/Log/ExceptionSerializer.php b/lib/private/Log/ExceptionSerializer.php index a792f966edaf3..fce1e7679fb55 100644 --- a/lib/private/Log/ExceptionSerializer.php +++ b/lib/private/Log/ExceptionSerializer.php @@ -42,6 +42,8 @@ use OCP\HintException; class ExceptionSerializer { + public const SENSITIVE_VALUE_PLACEHOLDER = '*** sensitive parameter replaced ***'; + public const methodsWithSensitiveParameters = [ // Session/User 'completeLogin', @@ -211,7 +213,7 @@ private function removeValuesFromArgs($args, $values) { $workArgs = []; foreach ($args as $arg) { if (in_array($arg, $values, true)) { - $arg = '*** sensitive parameter replaced ***'; + $arg = self::SENSITIVE_VALUE_PLACEHOLDER; } elseif (is_array($arg)) { $arg = $this->removeValuesFromArgs($arg, $values); } diff --git a/tests/lib/Log/ExceptionSerializerTest.php b/tests/lib/Log/ExceptionSerializerTest.php index f8cc6b77ee6c8..70ac80d13e3aa 100644 --- a/tests/lib/Log/ExceptionSerializerTest.php +++ b/tests/lib/Log/ExceptionSerializerTest.php @@ -62,7 +62,7 @@ public function testSerializer() { } catch (\Exception $e) { $serializedData = $this->serializer->serializeException($e); $this->assertSame(['Secret'], $secret); - $this->assertSame('*** sensitive parameters replaced ***', $serializedData['Trace'][0]['args'][0]); + $this->assertSame(ExceptionSerializer::SENSITIVE_VALUE_PLACEHOLDER, $serializedData['Trace'][0]['args'][0]); } } }