#26757 [Com]: session.save_path should default to %TEMP%
ID: 26757
Comment by: LarryJAdams at comcast dot net
Reported By: unknown at simplemachines dot org
Status: Closed
Bug Type: Session related
Operating System: Windows
PHP Version: 4CVS, 5CVS
New Comment:
I hear party favors going off somewhere...
Previous Comments:
------------------------------------------------------------------------
[2004-03-29 16:37:05] wez@php.net
Fixed in CVS.
The new default for save_path in upcoming releaess
will be the empty string, which causes the temporary
directory to be probed.
Thanks for your patch.
------------------------------------------------------------------------
[2004-03-29 09:56:43] unknown at simplemachines dot org
After I applied that patch, and changed session.save_path in my php.ini
to nothing, it did indeed work without trouble.
However, if I renamed php.ini to php_.ini, and tried it... it of course
went back to the default /tmp - but if the default is changed to NULL
without handling it, crashes will result...
My purpose here is to make it easier for people, mainly, to test PHP on
their own computers without running into funny errors. That means
default values, because some people will never read the manual....
which is why I did it as I did in my patch.
Thanks for replying,
-[Unknown]
------------------------------------------------------------------------
[2004-03-29 07:25:14] wez@php.net
Please try this patch against latest stable CVS:
http://www.php.net/~wez/session-files-fix-4.3.diff
I also have an equivalent for PHP 5.
This is slightly different from your patch, as it
doesn't obliterate non file-based session save paths.
------------------------------------------------------------------------
[2004-03-29 04:58:24] unknown at simplemachines dot org
I'm sorry, I just feel that with 47 votes (as of now) and even a
provided working patch, a comment from a developer would be nice.
Changing the category to "Session related" in the hopes that it not
only will be looked at there, but also will receive some notice. Was
previously a "Feature/Change Request".
Thanks again,
-[Unknown]
------------------------------------------------------------------------
[2004-02-17 03:14:45] unknown at simplemachines dot org
Maybe someone will read this if I say 4CVS, 5CVS instead of Irrelevant.
------------------------------------------------------------------------
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
http://bugs.php.net/26757
--
Edit this bug report at http://bugs.php.net/?id=26757&edit=1
Thread (8 messages)