@@ -66,7 +66,7 @@ |
||
| 66 | 66 | */ |
| 67 | 67 | public function register(): void |
| 68 | 68 | { |
| 69 | - $this->app->bind(SecurityPolicy::class, function (ContainerInterface $app) { |
|
| 69 | + $this->app->bind(SecurityPolicy::class, function(ContainerInterface $app) { |
|
| 70 | 70 | return new SecurityPolicy($app->get(Config::class)->get('security', [])); |
| 71 | 71 | }); |
| 72 | 72 | $this->app->bind(SecurityPolicyMiddleware::class); |
@@ -183,7 +183,7 @@ |
||
| 183 | 183 | */ |
| 184 | 184 | protected function shouldBeProcessed(ServerRequestInterface $request): bool |
| 185 | 185 | { |
| 186 | - //If no route has been match no need check for CSRF |
|
| 186 | + //If no route has been match no need check for CSRF |
|
| 187 | 187 | /** @var ?Route $route */ |
| 188 | 188 | $route = $request->getAttribute(Route::class); |
| 189 | 189 | if (!$route) { |
@@ -59,8 +59,7 @@ discard block |
||
| 59 | 59 | * @package Platine\Framework\Http\Middleware |
| 60 | 60 | * @template T |
| 61 | 61 | */ |
| 62 | -class SecurityPolicyMiddleware implements MiddlewareInterface |
|
| 63 | -{ |
|
| 62 | +class SecurityPolicyMiddleware implements MiddlewareInterface { |
|
| 64 | 63 | /** |
| 65 | 64 | * The SecurityPolicy instance |
| 66 | 65 | * @var SecurityPolicy |
@@ -71,8 +70,7 @@ discard block |
||
| 71 | 70 | * Create new instance |
| 72 | 71 | * @param SecurityPolicy $securityPolicy |
| 73 | 72 | */ |
| 74 | - public function __construct(SecurityPolicy $securityPolicy) |
|
| 75 | - { |
|
| 73 | + public function __construct(SecurityPolicy $securityPolicy) { |
|
| 76 | 74 | $this->securityPolicy = $securityPolicy; |
| 77 | 75 | } |
| 78 | 76 | |
@@ -150,7 +150,7 @@ discard block |
||
| 150 | 150 | return []; |
| 151 | 151 | } |
| 152 | 152 | |
| 153 | - $policy = new FeaturePermissionPolicy($config); |
|
| 153 | + $policy = new FeaturePermissionPolicy($config); |
|
| 154 | 154 | |
| 155 | 155 | return ['Permissions-Policy' => $policy->headers()]; |
| 156 | 156 | } |
@@ -167,7 +167,7 @@ discard block |
||
| 167 | 167 | return []; |
| 168 | 168 | } |
| 169 | 169 | |
| 170 | - $policy = new StrictTransportSecurityPolicy($config); |
|
| 170 | + $policy = new StrictTransportSecurityPolicy($config); |
|
| 171 | 171 | |
| 172 | 172 | return ['Strict-Transport-Security' => $policy->headers()]; |
| 173 | 173 | } |
@@ -185,7 +185,7 @@ discard block |
||
| 185 | 185 | return []; |
| 186 | 186 | } |
| 187 | 187 | |
| 188 | - $policy = new ClearSiteDataPolicy($config); |
|
| 188 | + $policy = new ClearSiteDataPolicy($config); |
|
| 189 | 189 | |
| 190 | 190 | return ['Clear-Site-Data' => $policy->headers()]; |
| 191 | 191 | } |
@@ -57,8 +57,7 @@ discard block |
||
| 57 | 57 | * @class SecurityPolicy |
| 58 | 58 | * @package Platine\Framework\Security |
| 59 | 59 | */ |
| 60 | -class SecurityPolicy |
|
| 61 | -{ |
|
| 60 | +class SecurityPolicy { |
|
| 62 | 61 | /** |
| 63 | 62 | * The configuration |
| 64 | 63 | * @var array<string, mixed> |
@@ -78,8 +77,7 @@ discard block |
||
| 78 | 77 | * Create new instance |
| 79 | 78 | * @param array<string, mixed> $configurations |
| 80 | 79 | */ |
| 81 | - public function __construct(array $configurations = []) |
|
| 82 | - { |
|
| 80 | + public function __construct(array $configurations = []) { |
|
| 83 | 81 | $this->configurations = $configurations; |
| 84 | 82 | } |
| 85 | 83 | |
@@ -52,8 +52,7 @@ |
||
| 52 | 52 | * @class StrictTransportSecurityPolicy |
| 53 | 53 | * @package Platine\Framework\Security\Policy |
| 54 | 54 | */ |
| 55 | -class StrictTransportSecurityPolicy extends AbstractPolicy |
|
| 56 | -{ |
|
| 55 | +class StrictTransportSecurityPolicy extends AbstractPolicy { |
|
| 57 | 56 | /** |
| 58 | 57 | * {@inheritdoc} |
| 59 | 58 | */ |
@@ -79,7 +79,7 @@ |
||
| 79 | 79 | $directives = array_intersect_key($config, $this->whitelist); |
| 80 | 80 | $values = array_filter($directives); |
| 81 | 81 | |
| 82 | - $results = array_map(function (string $directive) { |
|
| 82 | + $results = array_map(function(string $directive) { |
|
| 83 | 83 | if ($directive === 'execution-contexts') { |
| 84 | 84 | $directive = 'executionContexts'; |
| 85 | 85 | } |
@@ -52,8 +52,7 @@ |
||
| 52 | 52 | * @class ClearSiteDataPolicy |
| 53 | 53 | * @package Platine\Framework\Security\Policy |
| 54 | 54 | */ |
| 55 | -class ClearSiteDataPolicy extends AbstractPolicy |
|
| 56 | -{ |
|
| 55 | +class ClearSiteDataPolicy extends AbstractPolicy { |
|
| 57 | 56 | /** |
| 58 | 57 | * Clear Site Data white list directives. |
| 59 | 58 | * @var array<string, bool> |
@@ -52,8 +52,7 @@ discard block |
||
| 52 | 52 | * @class AbstractPolicy |
| 53 | 53 | * @package Platine\Framework\Security\Policy |
| 54 | 54 | */ |
| 55 | -abstract class AbstractPolicy |
|
| 56 | -{ |
|
| 55 | +abstract class AbstractPolicy { |
|
| 57 | 56 | /** |
| 58 | 57 | * The configuration |
| 59 | 58 | * @var array<string, mixed> |
@@ -64,8 +63,7 @@ discard block |
||
| 64 | 63 | * Create new instance |
| 65 | 64 | * @param array<string, mixed> $configurations |
| 66 | 65 | */ |
| 67 | - public function __construct(array $configurations = []) |
|
| 68 | - { |
|
| 66 | + public function __construct(array $configurations = []) { |
|
| 69 | 67 | $this->configurations = $configurations; |
| 70 | 68 | } |
| 71 | 69 | |
@@ -145,7 +145,7 @@ discard block |
||
| 145 | 145 | |
| 146 | 146 | $filtered = $this->filter($config, $whitelist); |
| 147 | 147 | |
| 148 | - return array_map(function (string $keyword) { |
|
| 148 | + return array_map(function(string $keyword) { |
|
| 149 | 149 | return sprintf('\'%s\'', $keyword); |
| 150 | 150 | }, $filtered); |
| 151 | 151 | } |
@@ -157,7 +157,7 @@ discard block |
||
| 157 | 157 | */ |
| 158 | 158 | public function schemes(array $schemes): array |
| 159 | 159 | { |
| 160 | - return array_map(function (string $scheme) { |
|
| 160 | + return array_map(function(string $scheme) { |
|
| 161 | 161 | $clean = trim($scheme); |
| 162 | 162 | |
| 163 | 163 | if (substr($clean, -1) === ':') { |
@@ -175,7 +175,7 @@ discard block |
||
| 175 | 175 | */ |
| 176 | 176 | public function nonces(array $nonces): array |
| 177 | 177 | { |
| 178 | - return array_map(function (string $nonce) { |
|
| 178 | + return array_map(function(string $nonce) { |
|
| 179 | 179 | $clean = trim($nonce); |
| 180 | 180 | |
| 181 | 181 | if (base64_decode($clean, true) === false) { |
@@ -222,7 +222,7 @@ discard block |
||
| 222 | 222 | { |
| 223 | 223 | $pluginTypes = $this->configurations['plugin-types'] ?? []; |
| 224 | 224 | |
| 225 | - $filtered = array_filter($pluginTypes, function (string $mime) { |
|
| 225 | + $filtered = array_filter($pluginTypes, function(string $mime) { |
|
| 226 | 226 | return preg_match('/^[a-z\-]+\/[a-z\-]+$/i', $mime); |
| 227 | 227 | }); |
| 228 | 228 | |
@@ -52,8 +52,7 @@ |
||
| 52 | 52 | * @class ContentSecurityPolicy |
| 53 | 53 | * @package Platine\Framework\Security\Policy |
| 54 | 54 | */ |
| 55 | -class ContentSecurityPolicy extends AbstractPolicy |
|
| 56 | -{ |
|
| 55 | +class ContentSecurityPolicy extends AbstractPolicy { |
|
| 57 | 56 | /** |
| 58 | 57 | * Content Security Policy white list directives. |
| 59 | 58 | * @var array<string, bool> |
@@ -111,7 +111,7 @@ |
||
| 111 | 111 | // ensure indexes are numerically |
| 112 | 112 | $urls = array_values($values); |
| 113 | 113 | |
| 114 | - return array_map(function (string $url) { |
|
| 114 | + return array_map(function(string $url) { |
|
| 115 | 115 | return sprintf('"%s"', $url); |
| 116 | 116 | }, $urls); |
| 117 | 117 | } |
@@ -52,8 +52,7 @@ |
||
| 52 | 52 | * @class FeaturePermissionPolicy |
| 53 | 53 | * @package Platine\Framework\Security\Policy |
| 54 | 54 | */ |
| 55 | -class FeaturePermissionPolicy extends AbstractPolicy |
|
| 56 | -{ |
|
| 55 | +class FeaturePermissionPolicy extends AbstractPolicy { |
|
| 57 | 56 | /** |
| 58 | 57 | * {@inheritdoc} |
| 59 | 58 | */ |