Skip to content

Commit 9a1aec9

Browse files
authored
Merge pull request #303 from brefphp/log-errors
Always log errors to stderr
2 parents 5d65fd4 + 5a5e7ee commit 9a1aec9

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

src/php-85.ini

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
1-
; On the CLI we want errors to be sent to stdout -> those will end up in CloudWatch
2-
; In FPM workers we don't want that, but that is overridden by Bref when it starts PHP-FPM
3-
display_errors=1
1+
; We want errors to be sent to stderr -> those will end up in CloudWatch
2+
display_errors=0
3+
log_errors=1
44

55
; Since PHP 7.4 the default value is E_ALL
66
; We override it to set the recommended configuration value for production.

src/php.ini

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
1-
; On the CLI we want errors to be sent to stdout -> those will end up in CloudWatch
2-
; In FPM workers we don't want that, but that is overridden by Bref when it starts PHP-FPM
3-
display_errors=1
1+
; We want errors to be sent to stderr -> those will end up in CloudWatch
2+
display_errors=0
3+
log_errors=1
44

55
; Since PHP 7.4 the default value is E_ALL
66
; We override it to set the recommended configuration value for production.

0 commit comments

Comments
 (0)