Hi all,
I was browsing bugs and found old feature request that can be fixed easily.
Request #35703 when session_name("123") consist only digits, should warning
https://bugs.php.net/bug.php?id=35703&thanks=1
session.name cannot be integer like string, since it will be initialized as
"long" key hash while session module trying to find as "string" key hash. I
don't understand why some users are willing to use digits only session
name, but problem is problem.
It's not intuitive, but this limitation is documented.
http://www.php.net/manual/en/function.session-name.php
If any one of you think this should be fix (i.e. Raise error for invalid
session name) I'll fix this. If not, I'll reply as "Won't fix".
Any comments?
--
Yasuo Ohgaki
yohgaki@ohgaki.net
Hi!
I was browsing bugs and found old feature request that can be fixed easily.
Request #35703 when session_name("123") consist only digits, should warning
https://bugs.php.net/bug.php?id=35703&thanks=1session.name cannot be integer like string, since it will be initialized as
"long" key hash while session module trying to find as "string" key hash. I
don't understand why some users are willing to use digits only session
name, but problem is problem.
Yes, I think we need to generate some message on that, given that such
session name would not actually work properly.
Stanislav Malyshev, Software Architect
SugarCRM: http://www.sugarcrm.com/
(408)454-6900 ext. 227
Hi Stas,
2013/6/29 Stas Malyshev smalyshev@sugarcrm.com
session.name cannot be integer like string, since it will be
initialized as
"long" key hash while session module trying to find as "string" key
hash. I
don't understand why some users are willing to use digits only session
name, but problem is problem.Yes, I think we need to generate some message on that, given that such
session name would not actually work properly.
I'll check session.name by adding INI_MH for it. Since it does not work at
all,
I'll raise E_WARNING for string only session.name.
Regards,
--
Yasuo Ohgaki
yohgaki@ohgaki.net