OpenConext / Stepup-SelfService

src/Surfnet/StepupSelfService/SelfServiceBundle/Security/Authentication/Token/SamlToken.php

Doc Comments +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
     /**
      * Returns the user credentials.
      *
-     * @return mixed The user credentials
+     * @return string The user credentials
      */
     public function getCredentials()
     {

src/Surfnet/StepupSelfService/SelfServiceBundle/Security/Authentication/AuthenticatedSessionStateHandler.php

Doc Comments +3 added lines

@@ -65,6 +65,7 @@ 
 
     /**
      * @param string $uri
+     * @return void
      */
     public function setCurrentRequestUri($uri);
 
@@ -76,6 +77,7 @@ 
     /**
      * Migrates the current session to a new session id while maintaining all
      * session attributes.
+     * @return void
      */
     public function migrate();
 
@@ -84,6 +86,7 @@ 
      *
      * Clears all session attributes and flashes and regenerates the
      * session and deletes the old session from persistence
+     * @return void
      */
     public function invalidate();
 }

src/Surfnet/StepupSelfService/SelfServiceBundle/Security/Authentication/SamlAuthenticationStateHandler.php

Doc Comments +2 added lines

@@ -27,6 +27,7 @@ 
 
     /**
      * @param string $requestId
+     * @return void
      */
     public function setRequestId($requestId);
 
@@ -37,6 +38,7 @@ 
 
     /**
      * Removes the requestId from the session
+     * @return void
      */
     public function clearRequestId();
 }

src/Surfnet/StepupSelfService/SelfServiceBundle/Exception/AssertionFailedException.php

Doc Comments +5 added lines

@@ -27,6 +27,11 @@
     private $constraints;
 
     // @codingStandardsIgnoreStart Compliance with beberlei/assert's invalid argument exception
+
+    /**
+     * @param string $message
+     * @param integer $code
+     */
     public function __construct($message, $code, $propertyPath = null, $value, array $constraints = array())
     {
         parent::__construct($message, $code);

src/Surfnet/StepupSelfService/SelfServiceBundle/Service/SecondFactorService.php

Doc Comments +6 added lines, -1 removed lines

@@ -129,6 +129,11 @@ 
                $vettedSecondFactors->getTotalItems() > 0;
     }
 
+    /**
+     * @param string $identityId
+     * @param string $state
+     * @param string $secondFactorId
+     */
     public function identityHasSecondFactorOfStateWithId($identityId, $state, $secondFactorId)
     {
         switch ($state) {
@@ -309,7 +314,7 @@ 
     }
 
     /**
-     * @param $identity
+     * @param \Surfnet\StepupMiddlewareClientBundle\Identity\Dto\Identity $identity
      * @param $allSecondFactors
      * @param $allowedSecondFactors
      * @param $maximumNumberOfRegistrations

src/Surfnet/StepupSelfService/SamlStepupProviderBundle/DependencyInjection/SurfnetStepupSelfServiceSamlStepupProviderExtension.php

Doc Comments +9 added lines

@@ -49,6 +49,9 @@ 
         }
     }
 
+    /**
+     * @param string $provider
+     */
     private function loadProviderConfiguration(
         $provider,
         array $configuration,
@@ -237,6 +240,9 @@ 
         $container->setDefinition('gssp.provider.' . $provider . '.metadata.factory', $metadataFactory);
     }
 
+    /**
+     * @param string $provider
+     */
     private function createRouteConfig($provider, $routeName)
     {
         // In the future, we ought to wrap this in an object.
@@ -247,6 +253,9 @@ 
         ];
     }
 
+    /**
+     * @param string $type
+     */
     private function validateDescriptions($descriptions, $appUrl, $provider, $type)
     {
         $regex ="/%%{$type}_link_start%%[a-zA-Z0-9 ]+%%{$type}_link_end%%/";

src/Surfnet/StepupSelfService/SelfServiceBundle/Value/GsspToken.php

Doc Comments +1 added lines, -1 removed lines

@@ -67,7 +67,7 @@
     }
 
     /**
-     * @return mixed
+     * @return string
      */
     public function getType()
     {

src/Surfnet/StepupSelfService/SelfServiceBundle/Mock/RemoteVetting/MockGateway.php

Doc Comments +7 added lines, -2 removed lines

@@ -141,7 +141,6 @@ 
     }
 
     /**
-     * @param string $samlRequest
      * @param string $fullRequestUri
      * @return SAML2AuthnRequest
      * @throws \Exception
@@ -343,7 +342,7 @@ 
 
     /**
      * @param Assertion $assertion
-     * @param $authnContextClassRef
+     * @param string $authnContextClassRef
      */
     private function addAuthenticationStatementTo(Assertion $assertion, $authnContextClassRef)
     {
@@ -398,6 +397,9 @@ 
         return $this->gatewayConfiguration->getIdentityProviderPublicKeyCertData();
     }
 
+    /**
+     * @param string $status
+     */
     private function isValidResponseStatus($status)
     {
         return in_array($status, [
@@ -408,6 +410,9 @@ 
         ]);
     }
 
+    /**
+     * @param string $subStatus
+     */
     private function isValidResponseSubStatus($subStatus)
     {
         return in_array($subStatus, [

src/Surfnet/StepupSelfService/SelfServiceBundle/Service/RemoteVetting/Encryption/IdentityWriterInterface.php

Doc Comments +1 added lines

@@ -24,6 +24,7 @@
      * Writes identity data to a data store. The data should be passed as a string.
      *
      * @param string $data
+     * @return void
      */
     public function write($data);
 }