KeepAliveRetryTransportWrapper::getTransport() - Code Metrics - Inspection of "Add retry wrapper to allow reconnect when the sock..." - bzikarsky/gelf-php - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#128)

by Alexandre

created 2021-01-04 16:24 UTC

KeepAliveRetryTransportWrapper::getTransport() A

↳ Parent: KeepAliveRetryTransportWrapper

Complexity

Conditions	1
Paths	1

Size

Total Lines	3
Code Lines	1

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	1
eloc	1
c	1
b	0
f	0
nc	1
nop	0
dl	0
loc	3
rs	10

<?php

namespace Gelf\Transport;

use Gelf\MessageInterface as Message;

class KeepAliveRetryTransportWrapper extends AbstractTransport
{
    /**
     * @const string
     */
    const NO_RESPONSE = "Graylog-Server didn't answer properly, expected 'HTTP/1.x 202 Accepted', response is ''";

    /**
     * @var HttpTransport
     */
    protected $transport;

    /**
     * KeepAliveRetryTransportWrapper constructor.
     *
     * @param HttpTransport $transport
     */
    public function __construct(HttpTransport $transport)
    {
        $this->transport = $transport;
    }

    /**
     * @return HttpTransport
     */
    public function getTransport()
    {
        return $this->transport;
    }

    /**
     * Sends a Message over this transport.
     *
     * @param Message $message
     *
     * @return int calls function to send message
     */
    public function send(Message $message)
    {
        try {
            return $this->transport->send($message);
        } catch (\RuntimeException $e) {
            if ($e->getMessage() !== self::NO_RESPONSE) {
                throw $e;
            }
            return $this->transport->send($message);
        }
    }
}


1			<?php
2
3			namespace Gelf\Transport;
4
5			use Gelf\MessageInterface as Message;
6
7			class KeepAliveRetryTransportWrapper extends AbstractTransport
8			{
9			/**
10			* @const string
11			*/
12			const NO_RESPONSE = "Graylog-Server didn't answer properly, expected 'HTTP/1.x 202 Accepted', response is ''";
13
14			/**
15			* @var HttpTransport
16			*/
17			protected $transport;
18
19			/**
20			* KeepAliveRetryTransportWrapper constructor.
21			*
22			* @param HttpTransport $transport
23			*/
24			public function __construct(HttpTransport $transport)
25			{
26			$this->transport = $transport;
27			}
28
29			/**
30			* @return HttpTransport
31			*/
32			public function getTransport()
33			{
34			return $this->transport;
35			}
36
37			/**
38			* Sends a Message over this transport.
39			*
40			* @param Message $message
41			*
42			* @return int calls function to send message
43			*/
44			public function send(Message $message)
45			{
46			try {
47			return $this->transport->send($message);
48			} catch (\RuntimeException $e) {
49			if ($e->getMessage() !== self::NO_RESPONSE) {
50			throw $e;
51			}
52			return $this->transport->send($message);
53			}
54			}
55			}
56

bzikarsky / gelf-php

Pull Request — master (#128)

KeepAliveRetryTransportWrapper::getTransport() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like