Skip to content
This repository was archived by the owner on Jan 29, 2020. It is now read-only.

Commit a260e84

Browse files
committed
Merge pull request #489 from gsdevme/patch-1
fix(WhoopsError): ensure the SCRIPT_NAME is set before attempting to use it
2 parents 852d228 + 3bc5640 commit a260e84

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

src/Middleware/WhoopsErrorResponseGenerator.php

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -92,10 +92,13 @@ private function prepareWhoopsHandler(ServerRequestInterface $request, PrettyPag
9292
$uri = $request->getAttribute('originalUri', false) ?: $request->getUri();
9393
$request = $request->getAttribute('originalRequest', false) ?: $request;
9494

95+
$serverParams = $request->getServerParams();
96+
$scriptName = isset($serverParams['SCRIPT_NAME']) ? $serverParams['SCRIPT_NAME'] : '';
97+
9598
$handler->addDataTable('Expressive Application Request', [
9699
'HTTP Method' => $request->getMethod(),
97100
'URI' => (string) $uri,
98-
'Script' => $request->getServerParams()['SCRIPT_NAME'],
101+
'Script' => $scriptName,
99102
'Headers' => $request->getHeaders(),
100103
'Cookies' => $request->getCookieParams(),
101104
'Attributes' => $request->getAttributes(),

0 commit comments

Comments
 (0)