refactor(config): Rename rate_limit to rate_limiter

- Updated configuration keys from 'rate_limit' to 'rate_limiter' for clarity
- Adjusted related references in Channel.php to maintain functionality
- Ensured consistency in naming for better code readability and understanding

Author: guanguans

config/exception-notify.php

@@ -52,11 +52,11 @@
     /**
      * The rate limit of same exception.
      */
-    'rate_limit' => [
-        'cache_store' => env('EXCEPTION_NOTIFY_RATE_LIMIT_CACHE_STORE'),
-        'key_prefix' => env('EXCEPTION_NOTIFY_RATE_LIMIT_KEY_PREFIX', 'exception-notify:rate-limit:'),
-        'max_attempts' => (int) env('EXCEPTION_NOTIFY_RATE_LIMIT_MAX_ATTEMPTS', config('app.debug') ? 50 : 1),
-        'decay_seconds' => (int) env('EXCEPTION_NOTIFY_RATE_LIMIT_DECAY_SECONDS', 300),
+    'rate_limiter' => [
+        'cache_store' => env('EXCEPTION_NOTIFY_RATE_LIMITER_CACHE_STORE'),
+        'key_prefix' => env('EXCEPTION_NOTIFY_RATE_LIMITER_KEY_PREFIX', 'exception-notify:rate-limit:'),
+        'max_attempts' => (int) env('EXCEPTION_NOTIFY_RATE_LIMITER_MAX_ATTEMPTS', config('app.debug') ? \PHP_INT_MAX : 1),
+        'decay_seconds' => (int) env('EXCEPTION_NOTIFY_RATE_LIMITER_DECAY_SECONDS', 300),
     ],
 
     /**

src/Channels/Channel.php

@@ -84,15 +84,7 @@ public function shouldReport(\Throwable $throwable): bool
      */
     private function shouldntReport(\Throwable $throwable): bool
     {
-        if ($this->shouldSkip($throwable)) {
-            return true;
-        }
-
-        return !$this->attempt(
-            $this->fingerprintFor($throwable),
-            config('exception-notify.rate_limit.max_attempts'),
-            config('exception-notify.rate_limit.decay_seconds')
-        );
+        return $this->shouldSkip($throwable) ? true : !$this->attempt($throwable);
     }
 
     private function shouldSkip(\Throwable $throwable): bool
@@ -107,25 +99,29 @@ private function shouldSkip(\Throwable $throwable): bool
     }
 
     /**
-     * @see \Illuminate\Foundation\Exceptions\Handler::shouldntReport()
-     * @see \Illuminate\Foundation\Exceptions\Handler::throttle()
-     * @see \Illuminate\Foundation\Exceptions\Handler::throttleUsing()
+     * @see RateLimiter::attempt()
+     * @see \Illuminate\Cache\RateLimiting\Limit
      */
-    private function fingerprintFor(\Throwable $throwable): string
+    private function attempt(\Throwable $throwable): bool
     {
-        return config('exception-notify.rate_limit.key_prefix').hash(
-            'xxh128',
-            implode(':', [$throwable->getFile(), $throwable->getLine(), $throwable->getCode()])
+        return with(new RateLimiter(Cache::store(config('exception-notify.rate_limiter.cache_store'))))->attempt(
+            $this->fingerprintFor($throwable),
+            config('exception-notify.rate_limiter.max_attempts'),
+            static fn (): bool => true,
+            config('exception-notify.rate_limiter.decay_seconds')
         );
     }
 
     /**
-     * @see RateLimiter::attempt
+     * @see \Illuminate\Foundation\Exceptions\Handler::shouldntReport()
+     * @see \Illuminate\Foundation\Exceptions\Handler::throttle()
+     * @see \Illuminate\Foundation\Exceptions\Handler::throttleUsing()
      */
-    private function attempt(string $key, int $maxAttempts, int $decaySeconds = 60): bool
+    private function fingerprintFor(\Throwable $throwable): string
     {
-        return (
-            new RateLimiter(Cache::store(config('exception-notify.rate_limit.cache_store', config('cache.default'))))
-        )->attempt($key, $maxAttempts, static fn (): bool => true, $decaySeconds);
+        return config('exception-notify.rate_limiter.key_prefix').hash(
+            'sha256',
+            implode(':', [$throwable->getFile(), $throwable->getLine(), $throwable->getCode()])
+        );
     }
 }