Great Tutorial

I think this check doesn't make sense

$user = $this->users[$connection->resourceId];

if( !isset($user){ return; }

because if $this->users doesn't have the specified key it will pop undefined offset before going to the next line including the check which will never evaluate to true in my opinion because if there's no error with the first line then $user will be surely set and having a non false value