Bug #77330 [Com]: session_id() no longer works inside custom SessionHandlerInterface

Edit report at https://bugs.php.net/bug.php?id=77330&edit=1 ID: 77330 Comment by: e6990620 at gmail dot com Reported by: e6990620 at gmail dot com Summary: session_id() no longer works inside custom SessionHandlerInterface Status: Assigned Type: Bug Package: Session related Operating System: Linux PHP Version: 7.3.0 Assigned To: yohgaki Block user comment: N Private report: N New Comment: I can confirm that the session.use_strict_mode directive is disabled in all PHP versions (though "strict mode" is actually enforced by the DemoBugSessionHandler code itself). To verify this I just appended this line at the end of the test script in 3v4l: echo ini_get('session.use_strict_mode') . PHP_EOL; and reran. Previous Comments: ------------------------------------------------------------------------ [2019-01-17 10:08:44] yohgaki@php.net Ok thanks. I'll have a look to see what is going on. ------------------------------------------------------------------------ [2019-01-17 10:04:53] nikic@php.net @yohgaki: I'm not sure I see what strict mode has to do with this. This change has been introduced in https://github.com/php/php-src/pull/2406. It's a new check, independent of strict mode. ------------------------------------------------------------------------ [2019-01-17 10:01:34] yohgaki@php.net Unless someone changed code and test, there is strict_mode phpt so I suppose stict_mode is not broken. Reporter, please verify. ------------------------------------------------------------------------ [2019-01-17 09:53:09] yohgaki@php.net While I've been proposed enabling stricrt_mode by default, but it's not enabled yet. Current implementation requires 2 session data read when strict mode is enabled, but this can be save to 1 read. This is rather simple change, but it requires save handler API change. ------------------------------------------------------------------------ [2019-01-17 09:50:23] yohgaki@php.net Your PHP 7.2 is enabling session.strict_mode=On while 7.3 is not. ------------------------------------------------------------------------ The remainder of the comments for this report are too long. To view the rest of the comments, please view the bug report online at https://bugs.php.net/bug.php?id=77330 -- Edit this bug report at https://bugs.php.net/bug.php?id=77330&edit=1