Target::fromConfig() - Code Metrics - Inspection of "Add HTTP basic auth option" - integer-net/callback-proxy - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#2)

by Fabian

created 2018-10-30 16:25 UTC

Target::fromConfig() A

↳ Parent: Target

Complexity

Conditions	4
Paths	4

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	10
CRAP Score	4

Importance

Changes

Metric	Value
dl	0
loc	16
ccs	10
cts	10
cp	1
rs	9.7333
c	0
b	0
f	0
cc	4
nc	4
nop	1
crap	4

<?php
declare(strict_types=1);

namespace IntegerNet\CallbackProxy;

use Psr\Http\Message\RequestInterface;
use Psr\Http\Message\ResponseInterface;
use Slim\Http\Uri;

class Target
{
    /**
     * @var \Psr\Http\Message\UriInterface
     */
    private $uri;

    /**
     * @var string[]
     */
    private $additionalHeaders;

    public function __construct(\Psr\Http\Message\UriInterface $uri, array $additionalHeaders = [])
    {
        $this->uri = $uri;
        $this->additionalHeaders = $additionalHeaders;
    }

    public static function fromConfig($config): self
    {
        if (is_string($config)) {
            return new self(Uri::createFromString($config));
        }
        if (!isset($config['uri'])) {
            throw new \InvalidArgumentException(
                '$config argument must be a string or an array containing an "uri" element'
            );
        }
        $additionalHeaders = [];
        if (isset($config['basic-auth'])) {
            $additionalHeaders['Authorization'] = 'Basic ' . \base64_encode($config['basic-auth']);
        }
        return new self(Uri::createFromString($config['uri']), $additionalHeaders);
    }

    public function applyToRequest(RequestInterface $request, string $action): RequestInterface
    {
        $request = $request->withUri($this->uri->withPath($this->uri->getPath() . $action));
        foreach ($this->additionalHeaders as $name => $value) {
            $request = $request->withHeader($name, $value);
        }
        return $request;
    }

    public function applyToResponse(ResponseInterface $response, string $action): ResponseInterface
    {
        return $response->withHeader(
            'X-Response-From',
            $this->uri->withPath($this->uri->getPath() . $action)->__toString()
        );
    }
}


1		<?php
2		declare(strict_types=1);
3
4		namespace IntegerNet\CallbackProxy;
5
6		use Psr\Http\Message\RequestInterface;
7		use Psr\Http\Message\ResponseInterface;
8		use Slim\Http\Uri;
9
10		class Target
11		{
12		/**
13		* @var \Psr\Http\Message\UriInterface
14		*/
15		private $uri;
16
17		/**
18		* @var string[]
19		*/
20		private $additionalHeaders;
21
22	48	public function __construct(\Psr\Http\Message\UriInterface $uri, array $additionalHeaders = [])
23		{
24	48	$this->uri = $uri;
25	48	$this->additionalHeaders = $additionalHeaders;
26	48	}
27
28	12	public static function fromConfig($config): self
29		{
30	12	if (is_string($config)) {
31	6	return new self(Uri::createFromString($config));
32		}
33	9	if (!isset($config['uri'])) {
34	3	throw new \InvalidArgumentException(
35	3	'$config argument must be a string or an array containing an "uri" element'
36		);
37		}
38	6	$additionalHeaders = [];
39	6	if (isset($config['basic-auth'])) {
40	6	$additionalHeaders['Authorization'] = 'Basic ' . \base64_encode($config['basic-auth']);
41		}
42	6	return new self(Uri::createFromString($config['uri']), $additionalHeaders);
43		}
44
45	42	public function applyToRequest(RequestInterface $request, string $action): RequestInterface
46		{
47	42	$request = $request->withUri($this->uri->withPath($this->uri->getPath() . $action));
48	42	foreach ($this->additionalHeaders as $name => $value) {
49	3	$request = $request->withHeader($name, $value);
50		}
51	42	return $request;
52		}
53
54	42	public function applyToResponse(ResponseInterface $response, string $action): ResponseInterface
55		{
56	42	return $response->withHeader(
57	42	'X-Response-From',
58	42	$this->uri->withPath($this->uri->getPath() . $action)->__toString()
59		);
60		}
61		}
62

integer-net / callback-proxy

Pull Request — master (#2)

Target::fromConfig() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like