Fix roles issue #51

Author: prophet777

Client/Auth/WebsocketAuthenticationProvider.php

@@ -52,6 +52,11 @@ public function __construct(
         $this->logger = null === $logger ? new NullLogger() : $logger;
     }
 
+    /**
+     * @param ConnectionInterface $connection
+     *
+     * @return TokenInterface
+     */
     protected function getToken(ConnectionInterface $connection)
     {
         $token = null;
@@ -78,14 +83,19 @@ protected function getToken(ConnectionInterface $connection)
     }
 
     /**
-     * @param TokenInterface $token
+     * @param ConnectionInterface $conn
+     *
+     * @return TokenInterface
+     *
+     * @throws StorageException
+     * @throws \Exception
      */
     public function authenticate(ConnectionInterface $conn)
     {
         if (1 === count($this->firewalls) && 'ws_firewall' === $this->firewalls[0]) {
             $this->logger->warning(sprintf(
-                'User firewall is not configured, we have set %s by default',
-                $this->firewalls[0])
+                    'User firewall is not configured, we have set %s by default',
+                    $this->firewalls[0])
             );
         }
 
@@ -114,21 +124,10 @@ public function authenticate(ConnectionInterface $conn)
         $conn->WAMP->clientStorageId = $identifier;
 
         $this->logger->info(sprintf(
-            '%s connected [%]',
-            $username,
-            $user instanceof UserInterface ? implode(', ', $user->getRoles()) : array()
+            '%s connected',
+            $username
         ), $loggerContext);
 
         return $token;
     }
-
-    /**
-     * @param TokenInterface $token
-     *
-     * @return bool
-     */
-    public function supports(TokenInterface $token)
-    {
-        return $token instanceof WebsocketToken;
-    }
 }

Client/Driver/PredisDriver.php

@@ -15,7 +15,7 @@ class PredisDriver implements DriverInterface
     protected $client;
 
     /**
-     * string $prefix
+     * string $prefix.
      */
     protected $prefix;
 

Event/ClientEventListener.php

@@ -92,9 +92,8 @@ public function onClientDisconnect(ClientEvent $event)
             $loggerContext['username'] = $username;
 
             $this->logger->info(sprintf(
-                '%s disconnected [%]',
-                $username,
-                $user instanceof UserInterface ? implode(', ', $user->getRoles()) : array()
+                '%s disconnected',
+                $username
             ), $loggerContext);
         } catch (ClientNotFoundException $e) {
             $this->logger->info('user timed out', $loggerContext);