Closed mfulb closed 2 months ago
Test Suite | Status | Result |
---|---|---|
Multiverse | :white_check_mark: | 7/7 passing |
SOAK | :white_check_mark: | 56/56 passing |
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 78.35%. Comparing base (
393ca93
) to head (f8a208b
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Starting with PHP 8.0 the
set_error_handler()
callback function signature drops the$errcontext
final parameter so it only contains 4 parameters. This caused usingnewrelic_notice_error()
as a callback handler for PHP 8+ to not work as the API call did not accept only 4 arguments. This PR adds support for this function signature. There are also numerous tests added that will hopefully caught this kind of issue in the future.