Nekland /
Woketo
This project does not seem to handle request data directly as such no vulnerable execution paths were found.
include, or for example
via PHP's auto-loading mechanism.
These results are based on our legacy PHP analysis, consider migrating to our new PHP analysis engine instead. Learn more
| 1 | <?php |
||
| 2 | |||
| 3 | /** |
||
| 4 | * This file is a part of Woketo package. |
||
| 5 | * |
||
| 6 | * (c) Nekland <[email protected]> |
||
| 7 | * |
||
| 8 | * For the full license, take a look to the LICENSE file |
||
| 9 | * on the root directory of this project |
||
| 10 | */ |
||
| 11 | |||
| 12 | namespace Nekland\Woketo\Server; |
||
| 13 | |||
| 14 | use Nekland\Woketo\Core\AbstractConnection; |
||
| 15 | use Nekland\Woketo\Exception\NoHandlerException; |
||
| 16 | use Nekland\Woketo\Exception\RuntimeException; |
||
| 17 | use Nekland\Woketo\Exception\WebsocketException; |
||
| 18 | use Nekland\Woketo\Http\Request; |
||
| 19 | use Nekland\Woketo\Http\Response; |
||
| 20 | use Nekland\Woketo\Rfc6455\Frame; |
||
| 21 | use Nekland\Woketo\Rfc6455\MessageProcessor; |
||
| 22 | use Nekland\Woketo\Rfc6455\Handshake\ServerHandshake; |
||
| 23 | use React\EventLoop\LoopInterface; |
||
| 24 | use React\Socket\ConnectionInterface; |
||
| 25 | |||
| 26 | class Connection extends AbstractConnection |
||
| 27 | { |
||
| 28 | 9 | public function __construct( |
|
| 29 | ConnectionInterface $socketStream, |
||
| 30 | callable $messageHandler, |
||
| 31 | LoopInterface $loop, |
||
| 32 | MessageProcessor $messageProcessor, |
||
| 33 | ServerHandshake $handshake = null |
||
| 34 | ) { |
||
| 35 | 9 | parent::__construct($messageProcessor, $loop, $handshake ?: new ServerHandshake); |
|
| 36 | 9 | $this->stream = $socketStream; |
|
| 37 | 9 | $this->initListeners(); |
|
| 38 | 9 | $this->handler = $messageHandler; |
|
| 39 | 9 | } |
|
| 40 | |||
| 41 | 9 | private function initListeners() |
|
| 42 | { |
||
| 43 | $this->stream->on('data', function ($data) { |
||
| 44 | 8 | $this->processData($data); |
|
| 45 | 9 | }); |
|
| 46 | $this->stream->once('end', function() { |
||
| 47 | 3 | if (!$this->handshakeDone) { |
|
| 48 | 1 | $this->logger->info('Disconnected but websocket didn\'t start.'); |
|
| 49 | 1 | return; |
|
| 50 | } |
||
| 51 | 2 | $this->getHandler()->onDisconnect($this); |
|
| 52 | 9 | }); |
|
| 53 | $this->stream->on('error', function ($data) { |
||
| 54 | $this->error($data); |
||
| 55 | 9 | }); |
|
| 56 | 9 | } |
|
| 57 | |||
| 58 | 8 | private function processData($data) |
|
| 59 | { |
||
| 60 | try { |
||
| 61 | 8 | if (!$this->handshakeDone) { |
|
| 62 | 8 | $this->processHandshake($data); |
|
| 63 | } else { |
||
| 64 | 2 | $this->processMessage($data); |
|
| 65 | } |
||
| 66 | |||
| 67 | 7 | return; |
|
| 68 | 1 | } catch (WebsocketException $e) { |
|
| 69 | $this->messageProcessor->close($this->stream); |
||
| 70 | $this->logger->notice('Connection to ' . $this->getIp() . ' closed with error : ' . $e->getMessage()); |
||
| 71 | $this->getHandler()->onError($e, $this); |
||
| 72 | 1 | } catch (NoHandlerException $e) { |
|
| 73 | 1 | $this->getLogger()->info(sprintf('No handler found for uri %s. Connection closed.', $this->uri)); |
|
| 74 | 1 | $this->close(Frame::CLOSE_WRONG_DATA); |
|
| 75 | } |
||
| 76 | 1 | } |
|
| 77 | |||
| 78 | /** |
||
| 79 | * This method build a message and buffer data in case of incomplete data. |
||
| 80 | * |
||
| 81 | * @param string $data |
||
| 82 | */ |
||
| 83 | 2 | protected function processMessage(string $data) |
|
| 84 | { |
||
| 85 | // It may be a timeout going (we were waiting for data), let's clear it. |
||
| 86 | 2 | if ($this->timeout !== null) { |
|
| 87 | $this->loop->cancelTimer($this->timeout); |
||
| 88 | $this->timeout = null; |
||
| 89 | } |
||
| 90 | |||
| 91 | 2 | foreach ($this->messageProcessor->onData($data, $this->stream, $this->currentMessage) as $message) { |
|
| 92 | 2 | $this->currentMessage = $message; |
|
| 93 | 2 | if ($this->currentMessage->isComplete()) { |
|
| 94 | // Sending the message through the woketo API. |
||
| 95 | 2 | switch($this->currentMessage->getOpcode()) { |
|
| 96 | case Frame::OP_TEXT: |
||
| 97 | 1 | $this->getHandler()->onMessage($this->currentMessage->getContent(), $this); |
|
| 98 | 1 | break; |
|
| 99 | case Frame::OP_BINARY: |
||
| 100 | 1 | $this->getHandler()->onBinary($this->currentMessage->getContent(), $this); |
|
| 101 | 1 | break; |
|
| 102 | } |
||
| 103 | 2 | $this->currentMessage = null; |
|
| 104 | |||
| 105 | } else { |
||
| 106 | // We wait for more data so we start a timeout. |
||
| 107 | $this->timeout = $this->loop->addTimer(Connection::DEFAULT_TIMEOUT, function () { |
||
| 108 | $this->logger->notice('Connection to ' . $this->getIp() . ' timed out.'); |
||
| 109 | $this->messageProcessor->timeout($this->stream); |
||
| 110 | }); |
||
| 111 | } |
||
| 112 | } |
||
| 113 | 2 | } |
|
| 114 | |||
| 115 | public function write($frame, int $opCode = Frame::OP_TEXT) |
||
| 116 | { |
||
| 117 | try { |
||
| 118 | $this->messageProcessor->write($frame, $this->stream, $opCode); |
||
| 119 | } catch (WebsocketException $e) { |
||
| 120 | throw new RuntimeException($e); |
||
| 121 | } |
||
| 122 | } |
||
| 123 | |||
| 124 | /** |
||
| 125 | * @param mixed $data |
||
| 126 | */ |
||
| 127 | protected function error($data) |
||
| 128 | { |
||
| 129 | $message = "A connectivity error occurred: " . $data; |
||
| 130 | $this->logger->error($message); |
||
| 131 | $this->getHandler()->onError(new WebsocketException($message), $this); |
||
| 132 | } |
||
| 133 | |||
| 134 | /** |
||
| 135 | * If it's a new client, we need to make some special actions named the handshake. |
||
| 136 | * |
||
| 137 | * @param string $data |
||
| 138 | */ |
||
| 139 | 8 | protected function processHandshake(string $data) |
|
| 140 | { |
||
| 141 | 8 | if ($this->handshakeDone) { |
|
| 142 | return; |
||
| 143 | } |
||
| 144 | |||
| 145 | 8 | $request = Request::create($data); |
|
| 146 | 8 | $this->handshake->verify($request); |
|
| 147 | 8 | $this->uri = $request->getUri(); |
|
| 148 | 8 | $response = Response::createSwitchProtocolResponse(); |
|
| 149 | 8 | $this->handshake->sign($response, $this->handshake->extractKeyFromRequest($request)); |
|
|
0 ignored issues
–
show
|
|||
| 150 | 8 | $response->send($this->stream); |
|
| 151 | |||
| 152 | 8 | $this->handshakeDone = true; |
|
| 153 | 8 | $this->getHandler()->onConnection($this); |
|
| 154 | 7 | } |
|
| 155 | |||
| 156 | /** |
||
| 157 | * @return string |
||
| 158 | */ |
||
| 159 | 5 | public function getIp() |
|
| 160 | { |
||
| 161 | 5 | return $this->stream->getRemoteAddress(); |
|
| 162 | } |
||
| 163 | } |
||
| 164 |
It seems like the method you are trying to call exists only in some of the possible types.
Let’s take a look at an example:
Available Fixes
Add an additional type-check:
Only allow a single type to be passed if the variable comes from a parameter: