Merge pull request #34 from strotgen/master

src/Middlewares/TreblleMiddleware.php

@@ -195,7 +195,9 @@ public function maskFields($data): array
                                     }
                                 }
                             } else {
-                                $data[$key] = str_repeat('*', strlen($value));
+                                $data[$key] = $value != null
+                                    ? str_repeat('*', strlen($value))
+                                    : $value;
                             }
                         }
                     }

tests/Middlewares/TreblleMiddlewareTest.php

@@ -0,0 +1,40 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Treblle\Tests\Middlewares;
+
+use Treblle\Middlewares\TreblleMiddleware;
+use Treblle\Tests\TestCase;
+
+class TreblleMiddlewareTest extends TestCase
+{
+    /**
+     * @var TreblleMiddleware
+     */
+    private $treblleMiddleware;
+
+    protected function setUp(): void
+    {
+        parent::setUp();
+
+        $this->treblleMiddleware = new TreblleMiddleware();
+    }
+
+    public function testRequestWithNullValueIsMaskedCorrectly(): void
+    {
+        $requestWithNullField = [
+            'cc' => null,
+            'otherValue' => 'something',
+            'password' => '1234',
+        ];
+
+        $maskedRequest = $this->treblleMiddleware->maskFields($requestWithNullField);
+
+        $this->assertEquals($maskedRequest, [
+            'cc' => null, // Should be left as null value
+            'otherValue' => 'something',
+            'password' => '****', // Should be masked
+        ]);
+    }
+}