beyondcode / laravel-websockets

src/Server/Logger/ConnectionLogger.php

<?php

namespace BeyondCode\LaravelWebSockets\Server\Logger;

use Ratchet\ConnectionInterface;

class ConnectionLogger extends Logger implements ConnectionInterface
{
    /** @var \Ratchet\ConnectionInterface */
    protected $connection;

    public static function decorate(ConnectionInterface $app): self
    {
        $logger = app(self::class);

        return $logger->setConnection($app);
    }

    public function setConnection(ConnectionInterface $connection)
    {
        $this->connection = $connection;

        return $this;
    }

    protected function getConnection()
    {
        return $this->connection;
    }

    public function send($data)
    {
        $socketId = $this->connection->socketId ?? null;

Accessing socketId on the interface Ratchet\ConnectionInterface suggest that you code against a concrete implementation. How about adding an instanceof check?

        $this->info("Connection id {$socketId} sending message {$data}");

        $this->connection->send($data);
    }

    public function close()
    {
        $this->warn("Connection id {$this->connection->socketId} closing.");

Accessing socketId on the interface Ratchet\ConnectionInterface suggest that you code against a concrete implementation. How about adding an instanceof check?

        $this->connection->close();
    }

    public function __set($name, $value)
    {
        return $this->connection->$name = $value;
    }

    public function __get($name)
    {
        return $this->connection->$name;
    }

    public function __isset($name)
    {
        return isset($this->connection->$name);
    }

    public function __unset($name)
    {
        unset($this->connection->$name);
    }
}