Skip to content

Commit

Permalink
Allow to use DebugHeaders without debugger (#151)
Browse files Browse the repository at this point in the history
  • Loading branch information
vjik authored Jan 20, 2025
1 parent 48981f7 commit 927c806
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions src/Debug/Middleware/DebugHeaders.php
Original file line number Diff line number Diff line change
Expand Up @@ -18,16 +18,16 @@
final class DebugHeaders implements MiddlewareInterface
{
public function __construct(
private Debugger $debugger,
private UrlGeneratorInterface $urlGenerator,
private ?Debugger $debugger = null,
) {
}

public function process(ServerRequestInterface $request, RequestHandlerInterface $handler): ResponseInterface
{
$response = $handler->handle($request);

if (!$this->debugger->isActive()) {
if ($this->debugger === null || !$this->debugger->isActive()) {
return $response;
}

Expand Down
2 changes: 1 addition & 1 deletion tests/Unit/Debug/Middleware/DebugHeadersTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ public function testHeaders(): void
$debugger->startup(new stdClass());
$expectedId = $debugger->getId();

$middleware = new DebugHeaders($debugger, $urlGenerator);
$middleware = new DebugHeaders($urlGenerator, $debugger);
$response = $middleware->process(new ServerRequest(), $this->createRequestHandler());

$this->assertSame($expectedId, $response->getHeaderLine('X-Debug-Id'));
Expand Down

0 comments on commit 927c806

Please sign in to comment.