[PHP 8.4] Deprecation notice inconsistencies with opcache enabled

[lebris]

Description

I am observing inconsistent behavior with deprecation notices using php 8.4.1 when opcache is enabled.
Some deprecation notice disapeared after having been displayed few times.

Code to reproduce :

<?php

declare(strict_types=1);

error_reporting(E_ALL);
ini_set('display_errors', true);

// php 8.3 deprecation : triggered every time
$a = 'blabla';
$a--;

// php 8.4 deprecation : triggered once
function implicitNullable(string $foo = null) {}

Here's the output when sending few requests (with apache2 in my case)

for i in {0..5}; do curl http://localhost ;done
<br />
<b>Deprecated</b>:  implicitNullable(): Implicitly marking parameter $foo as nullable is deprecated, the explicit nullable type must be used instead in <b>/var/www/html/index.php</b> on line <b>14</b><br />
<br />
<b>Deprecated</b>:  Decrement on non-numeric string has no effect and is deprecated in <b>/var/www/html/index.php</b> on line <b>10</b><br />
<br />
<b>Deprecated</b>:  Decrement on non-numeric string has no effect and is deprecated in <b>/var/www/html/index.php</b> on line <b>10</b><br />
<br />
<b>Deprecated</b>:  Decrement on non-numeric string has no effect and is deprecated in <b>/var/www/html/index.php</b> on line <b>10</b><br />
<br />
<b>Deprecated</b>:  Decrement on non-numeric string has no effect and is deprecated in <b>/var/www/html/index.php</b> on line <b>10</b><br />
<br />
<b>Deprecated</b>:  Decrement on non-numeric string has no effect and is deprecated in <b>/var/www/html/index.php</b> on line <b>10</b><br />
<br />
<b>Deprecated</b>:  Decrement on non-numeric string has no effect and is deprecated in <b>/var/www/html/index.php</b> on line <b>10</b><br />

Observed Behavior

• The "implicit nullable" deprecation appears only once (or twice in my app) and stops appearing.
• The "decrement on non-numeric string" deprecation appears on every request as expected.

Expected Behavior

• Both deprecation notices should appear consistently on every request.

Note : When disabling opcache, both deprecation notices are displayed consistently on every request.

Is this a bug or I am missing something ?

In this example I'm using php 8.4.1 from the official docker image php:8.4-apache and the default php configuration.

I made a repo here where you can find everything to reproduce easely : https://github.com/lebris/php8.4-deprecation-opcache

PHP Version

PHP 8.4.1

Operating System

No response

[Girgias]

The "implicit nullable" deprecation is a compile time deprecation, which only happens once when Opcache is used, as that is what opcache does.

Not sure if this can be fixed @iluuu1994 ?

[iluuu1994]

Opcache records and replays warnings (see zend_begin_record_errors()). I don't know why it doesn't work here.

[cmb69]

opcache.record_warnings is Off by default.

@lebris, does it work if you set opcache.record_warnings=1 in your php.ini?

[lebris]

With opcache.record_warnings=1 it works as expected. Thanks

for i in {0..10}; do curl -k http://localhost/ ; done
<br />
<b>Deprecated</b>:  implicitNullable(): Implicitly marking parameter $foo as nullable is deprecated, the explicit nullable type must be used instead in <b>/var/www/html/index.php</b> on line <b>14</b><br />
<br />
<b>Deprecated</b>:  Decrement on non-numeric string has no effect and is deprecated in <b>/var/www/html/index.php</b> on line <b>10</b><br />
<br />
<b>Deprecated</b>:  implicitNullable(): Implicitly marking parameter $foo as nullable is deprecated, the explicit nullable type must be used instead in <b>/var/www/html/index.php</b> on line <b>14</b><br />
<br />
<b>Deprecated</b>:  Decrement on non-numeric string has no effect and is deprecated in <b>/var/www/html/index.php</b> on line <b>10</b><br />
<br />
<b>Deprecated</b>:  implicitNullable(): Implicitly marking parameter $foo as nullable is deprecated, the explicit nullable type must be used instead in <b>/var/www/html/index.php</b> on line <b>14</b><br />
[...]

Is this behavior (requiring opcache.record_warnings=1) intentional since PHP 8.4 ?
Should it be enabled by default (at least in dev envs) to ensure consistent reporting of deprecation warnings ?

[cmb69]

To my knowledge, opcache.record_warnings is available as of PHP 8.4.0, and defaulted to Off since it was introduced. You may only notice this now, since both deprecations are new (as of PHP 8.4.0).

I'm fine with leaving this Off by default, but won't mind enabling it by default. If you want to change the default, please write to the internals mailing list.

[iluuu1994]

Ah ok, I didn't know this setting was off by default. It makes sense, little reason to throw deprecations that always occur on every request, that just hurts performance. I think this can be closed then.

Should it be enabled by default (at least in dev envs) to ensure consistent reporting of deprecation warnings ?

There's really no point, unless you just want to flood your logs more.