Req #52982 [Asn->Csd]: Expose php_session_active to userland via new function (patch included)

Edit report at https://bugs.php.net/bug.php?id=52982&edit=1 ID: 52982 Updated by: arpad@php.net Reported by: php dot net at kenman dot net Summary: Expose php_session_active to userland via new function (patch included) -Status: Assigned +Status: Closed Type: Feature/Change Request Package: Session related PHP Version: 5.3.3 Assigned To: arpad Block user comment: N Private report: N New Comment: This bug has been fixed in SVN. Snapshots of the sources are packaged every three hours; this change will be in the next snapshot. You can grab the snapshot at http://snaps.php.net/. For Windows: http://windows.php.net/snapshots/ Thank you for the report, and for helping us make PHP better. Previous Comments: ------------------------------------------------------------------------ [2011-08-29 21:29:15] arpad@php.net Automatic comment from SVN on behalf of arpad Revision: http://svn.php.net/viewvc/?view=revision&revision=315745 Log: Expose session status via new function, session_status (Req #52982) ------------------------------------------------------------------------ [2011-07-26 18:53:23] arpad@php.net The following patch has been added/updated: Patch Name: php-trunk-session-status-bools Revision: 1311706403 URL: https://bugs.php.net/patch-display.php?bug=52982&patch=php-trunk-session-status-bools&revision=1311706403 ------------------------------------------------------------------------ [2011-07-26 18:52:27] arpad@php.net The following patch has been added/updated: Patch Name: php-trunk-session-status Revision: 1311706347 URL: https://bugs.php.net/patch-display.php?bug=52982&patch=php-trunk-session-status&revision=1311706347 ------------------------------------------------------------------------ [2011-07-26 14:31:21] php dot net at kenman dot net Thank you very much, Arpad; my rudimentary C skills were barely enough for me to craft the patch (which I somehow managed to compile and use), and so I appreciate the background info. I would be happy with any solution that helps me to solve the problem! ------------------------------------------------------------------------ [2011-07-25 22:37:37] arpad@php.net If you're trying to establish whether it's safe to start your own session, this patch is deficient because PS(session_status) is not a boolean - it could also be php_session_disabled. I'd imagine robust code to handle this (rather unusual) case where you require a session but don't know if it's present, would be something like: if (session_is_available()) { if (!session_is_active()) { if (headers_sent()) { // bail out } else { session_start(); } } } else { // bail out } We could instead provide a session_status() function which corresponds to the internal enum (returning SESSION_DISABLED, SESSION_NONE or SESSION_ACTIVE.) I assume this is what Kalle was talking about WRT exposing the other values. Both options are trivial patches. The former seems friendlier but more prone to error. ------------------------------------------------------------------------ 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=52982 -- Edit this bug report at https://bugs.php.net/bug.php?id=52982&edit=1