Add ability to enable/disable HTTP Logging via .env variable

- new configuration option enabled in config/http-logger.php
- update the default log implementation to check the enabled configuration before logging requests

Author: marcell-ferenc

config/http-logger.php

@@ -2,6 +2,11 @@
 
 return [
 
+    /*
+     * Determine if the http-logger middleware should be enabled.
+     */
+    'enabled' => env('HTTP_LOGGER_ENABLED', true),
+
     /*
      * The log profile which determines whether a request should be logged.
      * It should implement `LogProfile`.

src/LogNonGetRequests.php

@@ -8,6 +8,10 @@ class LogNonGetRequests implements LogProfile
 {
     public function shouldLogRequest(Request $request): bool
     {
+        if (! config('http-logger.enabled')) {
+            return false;
+        }
+
         return in_array(strtolower($request->method()), ['post', 'put', 'patch', 'delete']);
     }
 }

tests/IntegrationTest.php

@@ -1,9 +1,18 @@
 <?php
 
+use function PHPUnit\Framework\assertFileDoesNotExist;
 use function PHPUnit\Framework\assertFileExists;
 
 it('logs an incoming request via the middleware', function () {
     $this->call('post', '/');
 
     assertFileExists($this->getLogFile());
 });
+
+it('doesnt log an incoming request when disabled', function () {
+    config(['http-logger.enabled' => false]);
+
+    $this->call('post', '/');
+
+    assertFileDoesNotExist($this->getLogFile());
+});

tests/LogNonGetRequestsTest.php

@@ -21,3 +21,13 @@
         $this->assertFalse($this->logProfile->shouldLogRequest($request), "{$method} should not be logged.");
     }
 });
+
+it('doesnt log when disabled', function () {
+    config(['http-logger.enabled' => false]);
+
+    foreach (['post', 'put', 'patch', 'delete'] as $method) {
+        $request = $this->makeRequest($method, $this->uri);
+
+        $this->assertFalse($this->logProfile->shouldLogRequest($request), "{$method} should not be logged.");
+    }
+});