Http\HttplugBundle\Collector\ProfileClient

Complexity

Total Complexity

13

Size/Duplication

Total Lines	127
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	7

Test Coverage

Coverage

40%

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	sendAsyncRequest()	0	15	1
A	sendRequest()	0	17	2
A	collectRequestInformations()	0	12	2
A	collectResponseInformations()	0	9	2
A	collectExceptionInformations()	0	12	3
A	__construct()	0	15	3

<?php

namespace Http\HttplugBundle\Collector;

use Http\Client\Common\FlexibleHttpClient;
use Http\Client\Exception\HttpException;
use Http\Client\HttpAsyncClient;
use Http\Client\HttpClient;
use Psr\Http\Message\RequestInterface;
use Psr\Http\Message\ResponseInterface;

/**
 * The ProfileClient decorates any client that implement both HttpClient and HttpAsyncClient interfaces to gather target
 * url and response status code.
 *
 * @author Fabien Bourigault <bourigaultfabien@gmail.com>
 *
 * @internal
 */
class ProfileClient implements HttpClient, HttpAsyncClient
{
    /**
     * @var HttpClient|HttpAsyncClient
     */
    private $client;

    /**
     * @var Collector
     */
    private $collector;

    /**
     * @var Formatter
     */
    private $formatter;

    /**
     * @param HttpClient|HttpAsyncClient $client    The client to profile. Client must implement both HttpClient and
     *                                              HttpAsyncClient interfaces.
     * @param Collector                  $collector
     * @param Formatter                  $formatter
     */
    public function __construct($client, Collector $collector, Formatter $formatter)
    {
        if (!($client instanceof HttpClient && $client instanceof HttpAsyncClient)) {
            throw new \RuntimeException(sprintf(
                '%s first argument must implement %s and %s. Consider using %s.',
                    __METHOD__,
                    HttpClient::class,
                    HttpAsyncClient::class,
                    FlexibleHttpClient::class
            ));
        }
        $this->client = $client;
        $this->collector = $collector;
        $this->formatter = $formatter;
    }

    /**
     * {@inheritdoc}
     */
    public function sendAsyncRequest(RequestInterface $request)
    {
        $stack = $this->collector->getCurrentStack();
        $this->collectRequestInformations($request, $stack);

It seems like $stack defined by $this->collector->getCurrentStack() on line 64 can also be of type boolean; however, Http\HttplugBundle\Collector\ProfileClient::collectRequestInformations() does only seem to accept null|object<Http\HttplugBundle\Collector\Stack>, maybe add an additional type check?

        return $this->client->sendAsyncRequest($request)->then(function (ResponseInterface $response) use ($stack) {

The method sendAsyncRequest does only exist in Http\Client\HttpAsyncClient, but not in Http\Client\HttpClient.

            $this->collectResponseInformations($response, $stack);

It seems like $stack defined by $this->collector->getCurrentStack() on line 64 can also be of type boolean; however, Http\HttplugBundle\Collector\ProfileClient::collectResponseInformations() does only seem to accept null|object<Http\HttplugBundle\Collector\Stack>, maybe add an additional type check?

            return $response;
        }, function (\Exception $exception) use ($stack) {
            $this->collectExceptionInformations($exception, $stack);

It seems like $stack defined by $this->collector->getCurrentStack() on line 64 can also be of type boolean; however, Http\HttplugBundle\Collector\ProfileClient::collectExceptionInformations() does only seem to accept null|object<Http\HttplugBundle\Collector\Stack>, maybe add an additional type check?

            throw $exception;
        });
    }

    /**
     * {@inheritdoc}
     */
    public function sendRequest(RequestInterface $request)
    {
        $stack = $this->collector->getCurrentStack();
        $this->collectRequestInformations($request, $stack);

It seems like $stack defined by $this->collector->getCurrentStack() on line 83 can also be of type boolean; however, Http\HttplugBundle\Collector\ProfileClient::collectRequestInformations() does only seem to accept null|object<Http\HttplugBundle\Collector\Stack>, maybe add an additional type check?

        try {
            $response = $this->client->sendRequest($request);

The method sendRequest does only exist in Http\Client\HttpClient, but not in Http\Client\HttpAsyncClient.

            $this->collectResponseInformations($response, $stack);

It seems like $stack defined by $this->collector->getCurrentStack() on line 83 can also be of type boolean; however, Http\HttplugBundle\Collector\ProfileClient::collectResponseInformations() does only seem to accept null|object<Http\HttplugBundle\Collector\Stack>, maybe add an additional type check?

            return $response;
        } catch (\Exception $e) {
            $this->collectExceptionInformations($e, $stack);

It seems like $stack defined by $this->collector->getCurrentStack() on line 83 can also be of type boolean; however, Http\HttplugBundle\Collector\ProfileClient::collectExceptionInformations() does only seem to accept null|object<Http\HttplugBundle\Collector\Stack>, maybe add an additional type check?

            throw $e;
        }
    }

    /**
     * @param RequestInterface $request
     * @param Stack|null       $stack
     */
    private function collectRequestInformations(RequestInterface $request, Stack $stack = null)
    {
        if (!$stack) {
            return;
        }

        $stack->setRequestTarget($request->getRequestTarget());
        $stack->setRequestMethod($request->getMethod());
        $stack->setRequestScheme($request->getUri()->getScheme());
        $stack->setRequestHost($request->getUri()->getHost());
        $stack->setClientRequest($this->formatter->formatRequest($request));
    }

    /**
     * @param ResponseInterface $response
     * @param Stack|null        $stack
     */
    private function collectResponseInformations(ResponseInterface $response, Stack $stack = null)
    {
        if (!$stack) {
            return;
        }

        $stack->setResponseCode($response->getStatusCode());
        $stack->setClientResponse($this->formatter->formatResponse($response));
    }

    /**
     * @param \Exception $exception
     * @param Stack|null $stack
     */
    private function collectExceptionInformations(\Exception $exception, Stack $stack = null)
    {
        if ($exception instanceof HttpException) {
            $this->collectResponseInformations($exception->getResponse(), $stack);
        }

        if (!$stack) {
            return;
        }

        $stack->setClientException($this->formatter->formatException($exception));
    }
}