faeez/mxmbsocket
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Repo. Project Releases Wiki Activity

Bug Fixes

Browse Source 
Fixed bug where WebSocket protocols were being created for every connection
Enabled Garbage Collection in server
This commit is contained in:
Chris Boden 2011-11-25 09:41:11 -05:00
parent d75113ec5e
commit 4de9caaa78
2 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
lib/Ratchet/Application/Server/App.php
View File

@ -70,6 +70,7 @@ class App implements ApplicationInterface {
$this->_connections[$host->getResource()] = new Connection($host); $this->_connections[$host->getResource()] = new Connection($host);
$this->_resources[] = $host->getResource(); $this->_resources[] = $host->getResource();
gc_enable();
set_time_limit(0); set_time_limit(0);
ob_implicit_flush(); ob_implicit_flush();

5
lib/Ratchet/Application/WebSocket/App.php
View File

@ -16,7 +16,6 @@ use Ratchet\Application\WebSocket\Version;
* @todo Make sure this works both ways (client/server) as stack needs to exist on client for framing * @todo Make sure this works both ways (client/server) as stack needs to exist on client for framing
* @todo Learn about closing the socket. A message has to be sent prior to closing - does the message get sent onClose event or CloseConnection command? * @todo Learn about closing the socket. A message has to be sent prior to closing - does the message get sent onClose event or CloseConnection command?
* @todo Consider chaning this class to a State Pattern. If a WS App interface is passed use different state for additional methods used * @todo Consider chaning this class to a State Pattern. If a WS App interface is passed use different state for additional methods used
* @todo I think I need to overhaul the architecture of this...more onus should be on the VersionInterfaces in case of changes...let them handle more decisions, not just parsing
*/ */
class App implements ApplicationInterface, ConfiguratorInterface { class App implements ApplicationInterface, ConfiguratorInterface {
/** /**
@ -206,8 +205,8 @@ class App implements ApplicationInterface, ConfiguratorInterface {
} else { } else {
$ns = __NAMESPACE__ . "\\Version\\{$name}"; $ns = __NAMESPACE__ . "\\Version\\{$name}";
if ($ns::isProtocol($headers)) { if ($ns::isProtocol($headers)) {
$this->_version[$name] = new $ns; $this->_versions[$name] = new $ns;
return $this->_version[$name]; return $this->_versions[$name];
} }
} }
} }