Skip to content

Commit 8b77bf2

Browse files
authored
Merge pull request #6333 from kenjis/fix-error_404
fix: 404 page might display Exception message in production environment
2 parents f67736f + 8fb0e7e commit 8b77bf2

File tree

3 files changed

+2
-7
lines changed

3 files changed

+2
-7
lines changed

app/Views/errors/html/error_404.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@
7373
<h1>404 - File Not Found</h1>
7474

7575
<p>
76-
<?php if (! empty($message) && $message !== '(null)') : ?>
76+
<?php if (ENVIRONMENT !== 'production') : ?>
7777
<?= nl2br(esc($message)) ?>
7878
<?php else : ?>
7979
Sorry! Cannot seem to find the page you were looking for.

phpstan-baseline.neon.dist

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -445,11 +445,6 @@ parameters:
445445
count: 1
446446
path: system/Database/Seeder.php
447447

448-
-
449-
message: "#^Expression on left side of \\?\\? is not nullable\\.$#"
450-
count: 1
451-
path: system/Debug/Exceptions.php
452-
453448
-
454449
message: "#^Property Config\\\\Exceptions\\:\\:\\$sensitiveDataInTrace \\(array\\) in isset\\(\\) is not nullable\\.$#"
455450
count: 1

system/Debug/Exceptions.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -269,7 +269,7 @@ protected function collectVars(Throwable $exception, int $statusCode): array
269269
'title' => get_class($exception),
270270
'type' => get_class($exception),
271271
'code' => $statusCode,
272-
'message' => $exception->getMessage() ?? '(null)',
272+
'message' => $exception->getMessage(),
273273
'file' => $exception->getFile(),
274274
'line' => $exception->getLine(),
275275
'trace' => $trace,

0 commit comments

Comments
 (0)