Endpoint::logResponse() - Code Metrics - Inspection of "Introduce 'HttpClientInterface' and implementation..." - RogierW/rw-acme-client - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#36)

by Rogier

created 2023-11-21 10:45 UTC

Endpoint::logResponse() A

↳ Parent: Endpoint

Complexity

Conditions	1
Paths	1

Size

Total Lines	8
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
eloc	6
c	0
b	0
f	0
dl	0
loc	8
rs	10
cc	1
nc	1
nop	4

<?php

namespace Rogierw\RwAcme\Endpoints;

use Rogierw\RwAcme\Api;
use Rogierw\RwAcme\Http\Response;
use Rogierw\RwAcme\Support\KeyId;

abstract class Endpoint
{
    public function __construct(protected Api $client)
    {
    }

    protected function createKeyId(string $accountUrl, string $url, ?array $payload = null): array
    {
        return KeyId::generate(
            $this->client->localAccount()->getPrivateKey(),
            $accountUrl,
            $url,
            $this->client->nonce()->getNew(),
            $payload
        );
    }

    protected function getAccountPrivateKey(): string
    {
        return $this->client->localAccount()->getPrivateKey();
    }

    protected function logResponse(string $level, string $message, Response $response, array $additionalContext = []): void
    {
        $this->client->logger($level, $message, array_merge([
            'url' => $response->getRequestedUrl(),
            'status' => $response->getHttpResponseCode(),
            'headers' => $response->getHeaders(),
            'body' => $response->getBody(),
        ], $additionalContext));
    }
}


1			<?php
2
3			namespace Rogierw\RwAcme\Endpoints;
4
5			use Rogierw\RwAcme\Api;
6			use Rogierw\RwAcme\Http\Response;
7			use Rogierw\RwAcme\Support\KeyId;
8
9			abstract class Endpoint
10			{
11			public function __construct(protected Api $client)
12			{
13			}
14
15			protected function createKeyId(string $accountUrl, string $url, ?array $payload = null): array
16			{
17			return KeyId::generate(
18			$this->client->localAccount()->getPrivateKey(),
19			$accountUrl,
20			$url,
21			$this->client->nonce()->getNew(),
22			$payload
23			);
24			}
25
26			protected function getAccountPrivateKey(): string
27			{
28			return $this->client->localAccount()->getPrivateKey();
29			}
30
31			protected function logResponse(string $level, string $message, Response $response, array $additionalContext = []): void
32			{
33			$this->client->logger($level, $message, array_merge([
34			'url' => $response->getRequestedUrl(),
35			'status' => $response->getHttpResponseCode(),
36			'headers' => $response->getHeaders(),
37			'body' => $response->getBody(),
38			], $additionalContext));
39			}
40			}
41

RogierW / rw-acme-client

Pull Request — master (#36)

Endpoint::logResponse() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like