Cleanup
This commit is contained in:
parent
7e58dccdb7
commit
6140c94a33
@ -8,6 +8,10 @@ CHANGELOG
|
||||
|
||||
---
|
||||
|
||||
* 0.2.5 (2013-??-??)
|
||||
|
||||
* Fixed Hixie-76 handshake bug
|
||||
|
||||
* 0.2.4 (2013-03-09)
|
||||
|
||||
* Support for Symfony 2.2 and Guzzle 2.3
|
||||
|
@ -88,7 +88,10 @@ class WsServer implements MessageComponentInterface {
|
||||
* {@inheritdoc}
|
||||
*/
|
||||
public function onMessage(ConnectionInterface $from, $msg) {
|
||||
if (true !== $from->WebSocket->established) {
|
||||
if (true === $from->WebSocket->established) {
|
||||
return $from->WebSocket->version->onMessage($this->connections[$from], $msg);
|
||||
}
|
||||
|
||||
if (isset($from->WebSocket->request)) {
|
||||
$from->WebSocket->request->getBody()->write($msg);
|
||||
} else {
|
||||
@ -113,13 +116,13 @@ class WsServer implements MessageComponentInterface {
|
||||
} catch (\UnderflowException $e) {
|
||||
return;
|
||||
}
|
||||
$response->setHeader('X-Powered-By', \Ratchet\VERSION);
|
||||
|
||||
// This needs to be refactored later on, incorporated with routing
|
||||
if ('' !== ($agreedSubProtocols = $this->getSubProtocolString($from->WebSocket->request->getTokenizedHeader('Sec-WebSocket-Protocol', ',')))) {
|
||||
$response->setHeader('Sec-WebSocket-Protocol', $agreedSubProtocols);
|
||||
}
|
||||
|
||||
$response->setHeader('X-Powered-By', \Ratchet\VERSION);
|
||||
$from->send((string)$response);
|
||||
|
||||
if (101 != $response->getStatusCode()) {
|
||||
@ -135,9 +138,6 @@ class WsServer implements MessageComponentInterface {
|
||||
return $this->_decorating->onOpen($upgraded);
|
||||
}
|
||||
|
||||
$from->WebSocket->version->onMessage($this->connections[$from], $msg);
|
||||
}
|
||||
|
||||
/**
|
||||
* {@inheritdoc}
|
||||
*/
|
||||
|
Loading…
Reference in New Issue
Block a user