Skip to content

Commit 52839f6

Browse files
committed
make placeholder a const for reuse
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
1 parent 79e899a commit 52839f6

2 files changed

Lines changed: 4 additions & 2 deletions

File tree

lib/private/Log/ExceptionSerializer.php

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,6 +42,8 @@
4242
use OCP\HintException;
4343

4444
class ExceptionSerializer {
45+
public const SENSITIVE_VALUE_PLACEHOLDER = '*** sensitive parameter replaced ***';
46+
4547
public const methodsWithSensitiveParameters = [
4648
// Session/User
4749
'completeLogin',
@@ -211,7 +213,7 @@ private function removeValuesFromArgs($args, $values) {
211213
$workArgs = [];
212214
foreach ($args as $arg) {
213215
if (in_array($arg, $values, true)) {
214-
$arg = '*** sensitive parameter replaced ***';
216+
$arg = self::SENSITIVE_VALUE_PLACEHOLDER;
215217
} elseif (is_array($arg)) {
216218
$arg = $this->removeValuesFromArgs($arg, $values);
217219
}

tests/lib/Log/ExceptionSerializerTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@ public function testSerializer() {
6262
} catch (\Exception $e) {
6363
$serializedData = $this->serializer->serializeException($e);
6464
$this->assertSame(['Secret'], $secret);
65-
$this->assertSame('*** sensitive parameters replaced ***', $serializedData['Trace'][0]['args'][0]);
65+
$this->assertSame(ExceptionSerializer::SENSITIVE_VALUE_PLACEHOLDER, $serializedData['Trace'][0]['args'][0]);
6666
}
6767
}
6868
}

0 commit comments

Comments
 (0)