fix typo in 'threshold', rename to 'limit'

Author: onlime

README.md

@@ -36,10 +36,10 @@ You may override its configuration in your `.env` - the following environment va
 - `HTTP_CLIENT_GLOBAL_LOGGER_OBFUSCATE_ENABLED` (bool)
 - `HTTP_CLIENT_GLOBAL_LOGGER_OBFUSCATE_REPLACEMENT` (string)
 - `HTTP_CLIENT_GLOBAL_LOGGER_TRIM_RESPONSE_BODY_ENABLED` (bool)
-- `HTTP_CLIENT_GLOBAL_LOGGER_TRIM_RESPONSE_BODY_TRESHOLD` (int)
+- `HTTP_CLIENT_GLOBAL_LOGGER_TRIM_RESPONSE_BODY_LIMIT` (int)
 - `HTTP_CLIENT_GLOBAL_LOGGER_TRIM_RESPONSE_BODY_CONTENT_TYPE_WHITELIST` (string)
 
-(look into `config/http-client-global-logger.php` for further configuration and explanation)
+(look into `config/http-client-global-logger.php` for defaults, further configuration, and explanation)
 
 ## Features
 

config/http-client-global-logger.php

@@ -121,7 +121,7 @@
     */
     'trim_response_body' => [
         'enabled' => env('HTTP_CLIENT_GLOBAL_LOGGER_TRIM_RESPONSE_BODY_ENABLED', false),
-        'treshold' => env('HTTP_CLIENT_GLOBAL_LOGGER_TRIM_RESPONSE_BODY_TRESHOLD', 200),
+        'limit' => env('HTTP_CLIENT_GLOBAL_LOGGER_TRIM_RESPONSE_BODY_LIMIT', 200),
         'content_type_whitelist' => explode(',', env(
             'HTTP_CLIENT_GLOBAL_LOGGER_TRIM_RESPONSE_BODY_CONTENT_TYPE_WHITELIST',
             ',application/json'

src/Listeners/LogResponseReceived.php

@@ -57,13 +57,13 @@ private function trimBody(Response $psrResponse): Response|MessageInterface
             return $psrResponse;
         }
 
-        $treshold = config('http-client-global-logger.trim_response_body.treshold');
+        $limit = config('http-client-global-logger.trim_response_body.limit');
 
-        // Check if the body size exceeds the treshold
-        return ($psrResponse->getBody()->getSize() <= $treshold)
+        // Check if the body size exceeds the limit
+        return ($psrResponse->getBody()->getSize() <= $limit)
             ? $psrResponse
             : $psrResponse->withBody(Utils::streamFor(
-                Str::limit($psrResponse->getBody(), $treshold)
+                Str::limit($psrResponse->getBody(), $limit)
             ));
     }
 }

tests/HttpClientLoggerTest.php

@@ -69,16 +69,16 @@ function setupLogger(): MockInterface
         'disabled' =>  [
             'config' => [
                 'enabled' => false,
-                'treshold' => 10,
+                'limit' => 10,
                 'content_type_whitelist' => ['application/json'],
             ],
             'contentType' => 'application/octet-stream',
             'shouldTrim' => false,
         ],
-        'below_treshold' => [
+        'below_limit' => [
             'config' => [
                 'enabled' => true,
-                'treshold' => 20,
+                'limit' => 20,
                 'content_type_whitelist' => ['application/json'],
             ],
             'contentType' => 'application/octet-stream',
@@ -87,7 +87,7 @@ function setupLogger(): MockInterface
         'content_type_whitelisted' => [
             'config' => [
                 'enabled' => true,
-                'treshold' => 10,
+                'limit' => 10,
                 'content_type_whitelist' => ['application/octet-stream'],
             ],
             'contentType' => 'application/octet-stream',
@@ -96,7 +96,7 @@ function setupLogger(): MockInterface
         'trim' => [
             'config' => [
                 'enabled' => true,
-                'treshold' => 10,
+                'limit' => 10,
                 'content_type_whitelist' => ['application/json'],
             ],
             'contentType' => 'application/octet-stream',
@@ -105,7 +105,7 @@ function setupLogger(): MockInterface
         'no_content_type_trim' => [
             'config' => [
                 'enabled' => true,
-                'treshold' => 10,
+                'limit' => 10,
                 'content_type_whitelist' => ['application/octet-stream'],
             ],
             'contentType' => '',
@@ -114,7 +114,7 @@ function setupLogger(): MockInterface
         'no_content_type_whitelisted' => [
             'config' => [
                 'enabled' => true,
-                'treshold' => 10,
+                'limit' => 10,
                 'content_type_whitelist' => ['', 'application/octet-stream'],
             ],
             'contentType' => '',