OpenConext / Stepup-Gateway

config/bootstrap.php

Spacing +3 added lines, -3 removed lines

@@ -2,7 +2,7 @@ 
 
 use Symfony\Component\Dotenv\Dotenv;
 
-require dirname(__DIR__).'/vendor/autoload.php';
+require dirname(__DIR__) . '/vendor/autoload.php';
 
 if (!class_exists(Dotenv::class)) {
     throw new LogicException('Please run "composer require symfony/dotenv" to load the ".env" files configuring the application.');
@@ -10,11 +10,11 @@ 
 
 // Load cached env vars if the .env.local.php file exists
 // Run "composer dump-env prod" to create it (requires symfony/flex >=1.2)
-if (is_array($env = @include dirname(__DIR__).'/.env.local.php') && (!isset($env['APP_ENV']) || ($_SERVER['APP_ENV'] ?? $_ENV['APP_ENV'] ?? $env['APP_ENV']) === $env['APP_ENV'])) {
+if (is_array($env = @include dirname(__DIR__) . '/.env.local.php') && (!isset($env['APP_ENV']) || ($_SERVER['APP_ENV'] ?? $_ENV['APP_ENV'] ?? $env['APP_ENV']) === $env['APP_ENV'])) {
     (new Dotenv(false))->populate($env);
 } else {
     // load all the .env files
-    (new Dotenv(false))->loadEnv(dirname(__DIR__).'/.env');
+    (new Dotenv(false))->loadEnv(dirname(__DIR__) . '/.env');
 }
 
 $_SERVER += $_ENV;

public/index.php

Spacing +1 added lines, -1 removed lines

@@ -4,7 +4,7 @@
 use Symfony\Component\ErrorHandler\Debug;
 use Symfony\Component\HttpFoundation\Request;
 
-require dirname(__DIR__).'/config/bootstrap.php';
+require dirname(__DIR__) . '/config/bootstrap.php';
 
 if ($_SERVER['APP_DEBUG']) {
     umask(0000);

tests/features/bootstrap/ServiceProviderContext.php

Spacing +3 added lines, -3 removed lines

@@ -200,7 +200,7 @@ 
         $request = Saml2AuthnRequest::createNew($authnRequest);
         $query = $request->buildRequestQuery();
 
-        $this->getSession()->visit($request->getDestination().'?'.$query);
+        $this->getSession()->visit($request->getDestination() . '?' . $query);
     }
 
     /**
@@ -256,7 +256,7 @@ 
         );
         $request = Saml2AuthnRequest::createNew($authnRequest);
         $query = $request->buildRequestQuery();
-        $this->getSession()->visit($authnRequest->getDestination().'?'.$query);
+        $this->getSession()->visit($authnRequest->getDestination() . '?' . $query);
     }
 
     /**
@@ -297,7 +297,7 @@ 
 
         $request = Saml2AuthnRequest::createNew($authnRequest);
         $query = $request->buildRequestQuery();
-        $this->getSession()->visit($request->getDestination().'?'.$query);
+        $this->getSession()->visit($request->getDestination() . '?' . $query);
     }
 
     /**

Indentation +3 added lines, -3 removed lines

@@ -125,7 +125,7 @@ 
         $publicKeyLoader->loadCertificateFile('/var/www/html/ci/certificates/sp.crt');
         $keys = $publicKeyLoader->getKeys();
         /**
- * @var Key $cert 
+         * @var Key $cert 
 */
         $cert = $keys->first();
 
@@ -146,7 +146,7 @@ 
         $publicKeyLoader->loadCertificateFile('/var/www/html/ci/certificates/idp.crt');
         $keys = $publicKeyLoader->getKeys();
         /**
- * @var Key $cert 
+         * @var Key $cert 
 */
         $cert = $keys->first();
 
@@ -325,7 +325,7 @@ 
     public function getIdentityProvider()
     {
         /**
- * @var RequestStack $stack 
+         * @var RequestStack $stack 
 */
 
         $stack = $this->kernel->getContainer()->get('request_stack');

Switch Indentation +27 added lines, -27 removed lines

@@ -186,20 +186,20 @@ 
             $this->loadPrivateKey(new PrivateKey('/var/www/html/ci/certificates/sp.key', 'default'))
         );
         switch ($loa) {
-        case "1":
-        case "2":
-        case "3":
-            $authnRequest->setRequestedAuthnContext(
-                ['AuthnContextClassRef' => ['http://stepup.example.com/assurance/sfo-level' . $loa]]
-            );
-            break;
-        case "self-asserted":
-            $authnRequest->setRequestedAuthnContext(
-                ['AuthnContextClassRef' => ['http://stepup.example.com/assurance/loa-self-asserted']]
-            );
-            break;
-        default:
-            throw new RuntimeException(sprintf('The specified LoA-%s is not supported', $loa));
+            case "1":
+            case "2":
+            case "3":
+                $authnRequest->setRequestedAuthnContext(
+                    ['AuthnContextClassRef' => ['http://stepup.example.com/assurance/sfo-level' . $loa]]
+                );
+                break;
+            case "self-asserted":
+                $authnRequest->setRequestedAuthnContext(
+                    ['AuthnContextClassRef' => ['http://stepup.example.com/assurance/loa-self-asserted']]
+                );
+                break;
+            default:
+                throw new RuntimeException(sprintf('The specified LoA-%s is not supported', $loa));
         }
         $request = Saml2AuthnRequest::createNew($authnRequest);
         $query = $request->buildRequestQuery();
@@ -284,19 +284,19 @@ 
         );
 
         switch ($loa) {
-        case "1":
-        case "2":
-        case "3":
-            $authnRequest->setRequestedAuthnContext(
-                ['AuthnContextClassRef' => ['http://stepup.example.com/assurance/level' . $loa]]
-            );
-            break;
-        case "self-asserted":
-            $authnRequest->setRequestedAuthnContext(
-                ['AuthnContextClassRef' => ['http://stepup.example.com/assurance/loa-self-asserted']]
-            );
-        default:
-            throw new RuntimeException(sprintf('The specified LoA-%s is not supported', $loa));
+            case "1":
+            case "2":
+            case "3":
+                $authnRequest->setRequestedAuthnContext(
+                    ['AuthnContextClassRef' => ['http://stepup.example.com/assurance/level' . $loa]]
+                );
+                break;
+            case "self-asserted":
+                $authnRequest->setRequestedAuthnContext(
+                    ['AuthnContextClassRef' => ['http://stepup.example.com/assurance/loa-self-asserted']]
+                );
+            default:
+                throw new RuntimeException(sprintf('The specified LoA-%s is not supported', $loa));
         }
 
         $request = Saml2AuthnRequest::createNew($authnRequest);

tests/src/Controller/IdentityProviderController.php

Spacing +1 added lines, -1 removed lines

@@ -104,7 +104,7 @@
     {
         $newAssertion = new Assertion();
         $newAssertion->setNotBefore(time());
-        $newAssertion->setNotOnOrAfter(time() + (60 * 5));//
+        $newAssertion->setNotOnOrAfter(time() + (60 * 5)); //
         $newAssertion->setAttributes(['urn:mace:dir:attribute-def:eduPersonTargetedID' => [NameID::fromArray($nameId)]]);
         $newAssertion->setIssuer('https://idp.stepup.example.com/');
         $newAssertion->setIssueInstant(time());

Indentation +1 added lines, -1 removed lines

@@ -239,7 +239,7 @@
         $keyLoader = new KeyLoader();
         $keyLoader->loadCertificateFile('/var/www/html/ci/certificates/idp.crt');
         /**
- * @var \SAML2\Certificate\X509 $publicKey 
+         * @var \SAML2\Certificate\X509 $publicKey 
 */
         $publicKey = $keyLoader->getKeys()->getOnlyElement();
 

ci/index.php

Spacing +1 added lines, -1 removed lines

@@ -3,7 +3,7 @@
 use App\Kernel;
 use Symfony\Component\HttpFoundation\Request;
 
-require dirname(__DIR__).'/config/bootstrap.php';
+require dirname(__DIR__) . '/config/bootstrap.php';
 
 $kernel = new Kernel('test', false);
 $request = Request::createFromGlobals();

src/Surfnet/StepupGateway/SecondFactorOnlyBundle/Saml/ResponseFactory.php

Spacing +1 added lines, -1 removed lines

@@ -66,7 +66,7 @@
         $this->hostedIdentityProvider    = $hostedIdentityProvider;
         $this->proxyStateHandler         = $proxyStateHandler;
         $this->assertionSigningService   = $assertionSigningService;
-        $this->currentTime = is_null($now) ? new DateTime('now', new DateTimeZone('UTC')): $now;
+        $this->currentTime = is_null($now) ? new DateTime('now', new DateTimeZone('UTC')) : $now;
     }
 
     /**

src/Surfnet/StepupGateway/ApiBundle/DependencyInjection/Configuration.php

Spacing +1 added lines, -1 removed lines

@@ -33,7 +33,7 @@
                     ->scalarNode('http_basic_realm')
                         ->defaultValue('Secure Gateway API')
                         ->validate()
-                            ->ifTrue(function ($realm) {
+                            ->ifTrue(function($realm) {
                                 return !is_string($realm) || empty($realm);
                             })
                             ->thenInvalid("Invalid HTTP Basic realm '%s'. Must be string and non-empty.")

src/Surfnet/StepupGateway/ApiBundle/Controller/SmsController.php

Spacing +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
             return new JsonResponse(['status' => 'OK']);
         }
 
-        $errors = array_map(function ($error) {
+        $errors = array_map(function($error) {
             return sprintf('%s (#%d)', $error['description'], $error['code']);
         }, $result->getRawErrors());
 

src/Surfnet/StepupGateway/SamlStepupProviderBundle/DependencyInjection/SurfnetStepupGatewaySamlStepupProviderExtension.php

Spacing +3 added lines, -3 removed lines

@@ -41,7 +41,7 @@ 
         $configuration = new Configuration();
         $config = $this->processConfiguration($configuration, $configs);
 
-        $loader = new Loader\YamlFileLoader($container, new FileLocator(__DIR__.'/../Resources/config'));
+        $loader = new Loader\YamlFileLoader($container, new FileLocator(__DIR__ . '/../Resources/config'));
         $loader->load('services.yml');
 
         $connectedServiceProviders = $container->getDefinition('gssp.allowed_sps');
@@ -136,7 +136,7 @@ 
         $hostedDefinition = $this->buildHostedEntityDefinition($provider, $configuration, $routes);
         $container->setDefinition('gssp.provider.' . $provider . '.hosted_entities', $hostedDefinition);
 
-        $hostedSpDefinition  = (new Definition())
+        $hostedSpDefinition = (new Definition())
             ->setClass('Surfnet\SamlBundle\Entity\ServiceProvider')
             ->setFactory([new Reference('gssp.provider.' . $provider . '.hosted_entities'), 'getServiceProvider'])
             ->setPublic(false);
@@ -189,7 +189,7 @@ 
      */
     private function createRemoteDefinition($provider, array $configuration, ContainerBuilder $container)
     {
-        $definition    = new Definition('Surfnet\SamlBundle\Entity\IdentityProvider', [
+        $definition = new Definition('Surfnet\SamlBundle\Entity\IdentityProvider', [
             [
                 'entityId'        => $configuration['entity_id'],
                 'ssoUrl'          => $configuration['sso_url'],