Http\HttplugBundle\Collector\ProfileClient

Complexity

Total Complexity

14

Size/Duplication

Total Lines	157
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	9

Test Coverage

Coverage

40%

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	sendAsyncRequest()	0	19	1
A	sendRequest()	0	20	2
A	collectRequestInformations()	0	12	2
A	collectResponseInformations()	0	10	2
A	collectExceptionInformations()	0	13	3
A	getStopwatchEventName()	0	4	1
A	__construct()	0	16	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;
use Symfony\Component\Stopwatch\Stopwatch;
use Symfony\Component\Stopwatch\StopwatchEvent;

/**
 * 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;

    /**
     * @var Stopwatch
     */
    private $stopwatch;

    /**
     * @param HttpClient|HttpAsyncClient $client    The client to profile. Client must implement both HttpClient and
     *                                              HttpAsyncClient interfaces.
     * @param Collector                  $collector
     * @param Formatter                  $formatter
     * @param Stopwatch                  $stopwatch
     */
    public function __construct($client, Collector $collector, Formatter $formatter, Stopwatch $stopwatch)
    {
        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;
        $this->stopwatch = $stopwatch;
    }

    /**
     * {@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 73 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?

        $event = $this->stopwatch->start($this->getStopwatchEventName($request));

        return $this->client->sendAsyncRequest($request)->then(

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

            function (ResponseInterface $response) use ($event, $stack) {
                $event->stop();
                $this->collectResponseInformations($response, $event, $stack);

It seems like $stack defined by $this->collector->getCurrentStack() on line 73 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 ($event, $stack) {
                $this->collectExceptionInformations($exception, $event, $stack);

It seems like $stack defined by $this->collector->getCurrentStack() on line 73 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 96 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?

        $event = $this->stopwatch->start($this->getStopwatchEventName($request));

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

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

            $event->stop();

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

It seems like $stack defined by $this->collector->getCurrentStack() on line 96 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) {
            $event->stop();
            $this->collectExceptionInformations($e, $event, $stack);

It seems like $stack defined by $this->collector->getCurrentStack() on line 96 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 StopwatchEvent    $event
     * @param Stack|null        $stack
     */
    private function collectResponseInformations(ResponseInterface $response, StopwatchEvent $event, Stack $stack = null)
    {
        if (!$stack) {
            return;
        }

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

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

        if (!$stack) {
            return;
        }

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

    /**
     * Generates the event name.
     *
     * @param RequestInterface $request
     *
     * @return string
     */
    private function getStopwatchEventName(RequestInterface $request)
    {
        return sprintf('%s %s', $request->getMethod(), $request->getUri()->__toString());
    }
}