Skip to content

Commit b59278e

Browse files
authored
Merge pull request #55 from OpenClassrooms/add-logging-on-security-disable
2 parents 4b4a7e8 + f8d9a1d commit b59278e

File tree

1 file changed

+8
-1
lines changed

1 file changed

+8
-1
lines changed

src/Interceptor/Impl/SecurityInterceptor.php

+8-1
Original file line numberDiff line numberDiff line change
@@ -11,15 +11,21 @@
1111
use OpenClassrooms\ServiceProxy\Interceptor\Contract\PrefixInterceptor;
1212
use OpenClassrooms\ServiceProxy\Model\Request\Instance;
1313
use OpenClassrooms\ServiceProxy\Model\Response\Response;
14+
use Psr\Log\LoggerInterface;
15+
use Psr\Log\NullLogger;
1416
use Symfony\Component\ExpressionLanguage\ExpressionLanguage;
1517

1618
final class SecurityInterceptor extends AbstractInterceptor implements PrefixInterceptor
1719
{
20+
private readonly LoggerInterface $logger;
21+
1822
public function __construct(
19-
iterable $handlers = [],
23+
iterable $handlers = [],
2024
private readonly ?SecurityInterceptorConfig $config = null,
25+
?LoggerInterface $logger = null,
2126
) {
2227
parent::__construct($handlers);
28+
$this->logger = $logger ?? new NullLogger();
2329
}
2430

2531
public function getPrefixPriority(): int
@@ -41,6 +47,7 @@ public function prefix(Instance $instance): Response
4147
;
4248

4349
if ($this->config?->bypassSecurity) {
50+
$this->logger->error('Security is bypassed.');
4451
return new Response();
4552
}
4653

0 commit comments

Comments
 (0)