Spatie\GuzzleRateLimiterMiddleware\RateLimiterMiddleware

Complexity

Total Complexity

4

Size/Duplication

Total Lines	43
Duplicated Lines	51.16 %

Coupling/Cohesion

Components	0
Dependencies	3

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	perSecond()	11	11	1
A	perMinute()	11	11	1
A	__invoke()	0	8	1

<?php

namespace Spatie\GuzzleRateLimiterMiddleware;

use Psr\Http\Message\RequestInterface;

class RateLimiterMiddleware

Since you have declared the constructor as private, maybe you should also declare the class as final.

{
    /** @var \Spatie\GuzzleRateLimiterMiddleware\RateLimiter */
    protected $rateLimiter;

    private function __construct(RateLimiter $rateLimiter)
    {
        $this->rateLimiter = $rateLimiter;
    }

    public static function perSecond(int $limit, Store $store = null): RateLimiterMiddleware

This method seems to be duplicated in your project.

    {
        $rateLimiter = new RateLimiter(
            $limit,
            RateLimiter::TIME_FRAME_SECOND,
            $store ?? new InMemoryStore(),
            new SleepDeferrer()
        );

        return new static($rateLimiter);
    }

    public static function perMinute(int $limit, Store $store = null): RateLimiterMiddleware

This method seems to be duplicated in your project.

    {
        $rateLimiter = new RateLimiter(
            $limit,
            RateLimiter::TIME_FRAME_MINUTE,
            $store ?? new InMemoryStore(),
            new SleepDeferrer()
        );

        return new static($rateLimiter);
    }

    public function __invoke(callable $handler)
    {
        return function (RequestInterface $request, array $options) use ($handler) {
            return $this->rateLimiter->handle(function () use ($request, $handler, $options) {
                return $handler($request, $options);
            });
        };
    }
}