Skip to content

Conversation

@jrfnl
Copy link
Member

@jrfnl jrfnl commented Oct 25, 2025

. Added fatal_error_backtraces to control whether fatal errors should include
a backtrace.
RFC: https://wiki.php.net/rfc/error_backtraces_v2

This commit accounts for detecting the ini setting.

Note: the PR for the RFC also mentions the following:

It defaults to E_FATAL_ERRORS, which I've also exposed as a userland constant.

However, by the looks of it, this new constant didn't make it into the final implementation (and isn't mentioned in the RFC either), so I've not updated the NewConstants sniff.

Refs:

Related to #1849

…ective (RFC)

> . Added fatal_error_backtraces to control whether fatal errors should include
>   a backtrace.
>   RFC: https://wiki.php.net/rfc/error_backtraces_v2

This commit accounts for detecting the ini setting.

Note: the PR for the RFC also mentions the following:
> It defaults to `E_FATAL_ERRORS`, which I've also exposed as a userland constant.

However, by the looks of it, this new constant didn't make it into the final implementation (and isn't mentioned in the RFC either), so I've not updated the `NewConstants` sniff.

Refs:
* https://wiki.php.net/rfc/error_backtraces_v2
* https://github.com/php/php-src/blob/34721745833b4c9e69ed22f1b17c4c1c421f7179/UPGRADING#L946-L948
* php/php-src 17056
* php/php-src@0a14ab1

Related to 1849
@jrfnl jrfnl added this to the 10.0.0-alpha2 milestone Oct 25, 2025
@jrfnl jrfnl requested a review from wimg October 25, 2025 23:13
@jrfnl jrfnl added Type: enhancement PR: quick merge PR only contains relatively simple changes PR: ready for review labels Oct 25, 2025
@wimg wimg merged commit 04e12a3 into develop Oct 27, 2025
76 checks passed
@wimg wimg deleted the php-8.5/newinidirectives-fatal-error-backtraces branch October 27, 2025 20:27
@github-actions github-actions bot removed PR: ready for review PR: quick merge PR only contains relatively simple changes labels Oct 27, 2025
@jrfnl jrfnl added the PHP: 8.5 label Nov 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants