ReleaseProtection\Http\Middleware\VerifyFirstPartyClientIp

Complexity

Total Complexity

3

Size/Duplication

Total Lines	28
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	handle()	0	16	3

<?php


namespace ReleaseProtection\Http\Middleware;

use Closure;

class VerifyFirstPartyClientIp
{

    /**
     * Handle an incoming request.
     *
     * @param \Illuminate\Http\Request $request
     * @param Closure $next
     * @param string $type
     *
     * @return mixed
     */
    public function handle($request, Closure $next, $type = 'default')
    {
        $allowIPs = config('release-protection.ip.ips.' . $type, config('release-protection.ip.ips.default'));
        if (
            app()->environment(

The method environment() does not exist on Illuminate\Container\Container. Are you sure you never get this type here, but always one of the subclasses?

                config('release-protection.ip.envs.' . $type, config('release-protection.ip.envs.default'))
            ) &&
            !in_array($request->ip(), $allowIPs)
        ) {
            abort(
                config('release-protection.ip.status.restricted.' . $type, config('release-protection.ip.status.restricted.default')),
                __(config('release-protection.ip.msg.restricted.' . $type, config('release-protection.ip.msg.restricted.default')))

It seems like __(config('release-protection.ip.msg.restricted.' . $type, config('release-protection.ip.msg.restricted.default'))) can also be of type array; however, parameter $message of abort() does only seem to accept string, maybe add an additional type check?

            );
        }

        return $next($request);
    }
}