ApiLoggerMiddleware::handleResponse() - Code Metrics - Inspection of "Merge branch 'release/0.5.0'" - nixilla/api-logger-bundle - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( cc0ba2...442efb )

by Janusz

created 2019-05-22 16:01 UTC

ApiLoggerMiddleware::handleResponse() A

↳ Parent: ApiLoggerMiddleware

Complexity

Conditions	3
Paths	2

Size

Total Lines	24
Code Lines	15

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	15
CRAP Score	3

Importance

Changes

Metric	Value
eloc	15
dl	0
loc	24
ccs	15
cts	15
cp	1
rs	9.7666
c	0
b	0
f	0
cc	3
nc	2
nop	3
crap	3

<?php

namespace Nixilla\Api\LoggerBundle\Middleware;

use Buzz\Middleware\MiddlewareInterface;
use Nixilla\Api\LoggerBundle\Logger\ApiInterface;
use Psr\Http\Message\RequestInterface;
use Psr\Http\Message\ResponseInterface;

class ApiLoggerMiddleware implements MiddlewareInterface
{
    /** @var ApiInterface */
    private $logger;

    /** @var float */
    private $startTime;

    /**
     * ApiLoggerMiddleware constructor.
     *
     * @param ApiInterface $logger
     */
    public function __construct(ApiInterface $logger)
    {
        $this->logger = $logger;
    }

    public function handleRequest(RequestInterface $request, callable $next)
    {
        $this->startTime = microtime(true);

        return $next($request);

    }

    public function handleResponse(RequestInterface $request, ResponseInterface $response, callable $next)
    {
        $time = microtime(true) - $this->startTime;

        if($request->getHeader('Content-Type') == 'application/json')

        {
            $payload = json_decode($request->getBody(), true) ?: [];
        }
        else {
            parse_str($request->getBody(), $payload);
        }

        $this->logger->logCall(
            sprintf('%s://%s', $request->getUri()->getScheme(), $request->getUri()->getHost()),
            $request->getUri()->getPath(),
            $request->getMethod(),
            $time,
            $request->getHeaders(),
            $payload,
            $response->getHeaders(),
            $response->getBody()->getContents()
        );

        return $next($request, $response);
    }
}


1		<?php
2
3		namespace Nixilla\Api\LoggerBundle\Middleware;
4
5		use Buzz\Middleware\MiddlewareInterface;
6		use Nixilla\Api\LoggerBundle\Logger\ApiInterface;
7		use Psr\Http\Message\RequestInterface;
8		use Psr\Http\Message\ResponseInterface;
9
10		class ApiLoggerMiddleware implements MiddlewareInterface
11		{
12		/** @var ApiInterface */
13		private $logger;
14
15		/** @var float */
16		private $startTime;
17
18		/**
19		* ApiLoggerMiddleware constructor.
20		*
21		* @param ApiInterface $logger
22		*/
23	4	public function __construct(ApiInterface $logger)
24		{
25	4	$this->logger = $logger;
26	4	}
27
28	1	public function handleRequest(RequestInterface $request, callable $next)
29		{
30	1	$this->startTime = microtime(true);
31
32	1	return $next($request);
33
34		}
35
36	2	public function handleResponse(RequestInterface $request, ResponseInterface $response, callable $next)
37		{
38	2	$time = microtime(true) - $this->startTime;
39
40	2	if($request->getHeader('Content-Type') == 'application/json')
		0 ignored issues – show introduced 2019-05-22 15:30 UTC by Report Bug Copy Issue Report The condition `$request->getHeader('Con...) == 'application/json'` is always `false`. Loading history...
41		{
42	1	$payload = json_decode($request->getBody(), true) ?: [];
43		}
44		else {
45	1	parse_str($request->getBody(), $payload);
46		}
47
48	2	$this->logger->logCall(
49	2	sprintf('%s://%s', $request->getUri()->getScheme(), $request->getUri()->getHost()),
50	2	$request->getUri()->getPath(),
51	2	$request->getMethod(),
52	2	$time,
53	2	$request->getHeaders(),
54	2	$payload,
55	2	$response->getHeaders(),
56	2	$response->getBody()->getContents()
57		);
58
59	2	return $next($request, $response);
60		}
61		}
62

nixilla / api-logger-bundle

Push — master ( cc0ba2...442efb )

ApiLoggerMiddleware::handleResponse() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like