Yiisoft\Yii\Debug\Collector\HttpClientInterfaceProxy::sendRequest()

Complexity

Conditions	1
Paths	2

Size

Total Lines	18
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function sendRequest(RequestInterface $request): ResponseInterface
    {
        /** @psalm-var array{file: string, line: int} $callStack */
        $callStack = debug_backtrace()[0];

        $uniqueId = random_bytes(36);
        $startTime = microtime(true);
        $this->collector->collect($request, $startTime, $callStack['file'] . ':' . $callStack['line'], $uniqueId);

It seems like $startTime can also be of type string; however, parameter $startTime of Yiisoft\Yii\Debug\Collector\HttpClientCollector::collect() does only seem to accept double, maybe add an additional type check?

        $response = null;

The assignment to $response is dead and can be removed.

        try {
            $response = $this->decorated->sendRequest($request);
        } finally {
            $endTime = microtime(true);
            $this->collector->collectTotalTime($response, $endTime, $uniqueId);

It seems like $endTime can also be of type string; however, parameter $endTime of Yiisoft\Yii\Debug\Collector\HttpClientCollector::collectTotalTime() does only seem to accept double, maybe add an additional type check?

        }

        return $response;
    }