Skip to content

config: change the default error reporting level in production #8219

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
kenjis merged 4 commits into from
Dec 3, 2023
Merged

config: change the default error reporting level in production #8219

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
d7f566c
config: change error_reporting to -1 in production
kenjis Nov 15, 2023
536b129
docs: add changelog and upgrade
kenjis Nov 15, 2023
02d525b
docs: add previous error_reporting() as comment
kenjis Nov 15, 2023
017a5c4
config: change to "E_ALL & ~E_DEPRECATED"
kenjis Nov 25, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 3 additions & 1 deletion app/Config/Boot/production.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,8 +9,10 @@
|
| If you set 'display_errors' to '1', CI4's detailed error report will show.
*/
error_reporting(E_ALL & ~E_DEPRECATED);
// If you want to suppress more types of errors.
// error_reporting(E_ALL & ~E_NOTICE & ~E_DEPRECATED & ~E_STRICT & ~E_USER_NOTICE & ~E_USER_DEPRECATED);
ini_set('display_errors', '0');
error_reporting(E_ALL & ~E_NOTICE & ~E_DEPRECATED & ~E_STRICT & ~E_USER_NOTICE & ~E_USER_DEPRECATED);

/*
|--------------------------------------------------------------------------
Expand Down
3 changes: 3 additions & 0 deletions user_guide_src/source/changelogs/v4.5.0.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -313,6 +313,9 @@ Changes
- **Config:**
- ``Config\Feature::$multipleFilters`` has been removed, because now
:ref:`multiple-filters` are always enabled.
- The default error level in the production environment
(**app/Config/Boot/production.php**) has been changed to ``E_ALL & ~E_DEPRECATED``
to match the default **php.ini** for production.
- **RouteCollection:** The HTTP method keys in the protected property ``$routes``
has been fixed from lowercase to uppercase.

Expand Down
2 changes: 2 additions & 0 deletions user_guide_src/source/installation/upgrade_450.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -255,6 +255,8 @@ The ``'toolbar'`` in the ``$global['after']`` was removed.
Others
^^^^^^

- app/Config/Boot/production.php
- The default error level to ``error_reporting()`` has been changed to ``E_ALL & ~E_DEPRECATED``.
- app/Config/Database.php
- The default value of ``charset`` in ``$default`` has been change to ``utf8mb4``.
- The default value of ``DBCollat`` in ``$default`` has been change to ``utf8mb4_general_ci``.
Expand Down