Edit report at https://bugs.php.net/bug.php?id=60179&edit=1
ID: 60179
Updated by: f...@php.net
Reported by: dbetz at df dot eu
Summary: wrong output with php_flag boolean
Status: Assigned
Type: Bug
Package: FPM related
Operating System: Gentoo Linux
PHP Version: 5.3.8
Assigned To: fat
Block user comment: N
Private report: N
New Comment:
hum ... not sure to understand your problem afterall :)
without the patch:
If I set php_flag[session.auto_start]=On
ini_get("session.auto_start") returns "On"
and session are created for each page (auto start is on as asked)
If I set php_flag[session.auto_start]=Off
ini_get("session.auto_start") returns "Off"
and session are NOT created for each page (auto start is off as asked)
For me this works as expected.
Do you have the same behavior ?
Previous Comments:
------------------------------------------------------------------------
[2011-10-31 13:52:48] dbetz at df dot eu
Hi fat,
i have tested the latest snapshot with the patch and have changed the config to
php_flag[session.auto_start] = Off
But the problem is still here.
Looking into fpm_conf.c at: static char *fpm_conf_set_array(... :
if (convert_to_bool) {
char *err = fpm_conf_set_boolean(value, &subconf, 0);
if (err) return err;
kv->value = strdup(b ? "On" : "Off");
}
It looks like kv->value is either string "On" or "Off" or am i wrong ?
------------------------------------------------------------------------
[2011-10-31 13:18:55] f...@php.net
1- On wich version did you apply the patch ? Please test the patch on the
lastest 5.3.x snapshot.
2- you should not surrender the parameter with double quote:
php_flag["session.auto_start"] = Off
should be
php_flag[session.auto_start] = Off
Can you get rid of the bug. On my side, I have the last 5.3.x snapshot and it
works great. I can't reproduce your problem.
------------------------------------------------------------------------
[2011-10-31 13:11:01] dbetz at df dot eu
Hello,
sorry, the problem still exists.
session.auto_start is : Off
Some code here .....
With cli:
session.auto_start is : 0
------------------------------------------------------------------------
[2011-10-31 13:02:09] f...@php.net
Can you please test the patch attached ?
If you can test it quickly I can integrate it before 5.3.9 is going RC1.
thx
++ fat
------------------------------------------------------------------------
[2011-10-31 13:00:41] f...@php.net
The following patch has been added/updated:
Patch Name: fpm-bug60179.patch
Revision: 1320066041
URL:
https://bugs.php.net/patch-display.php?bug=60179&patch=fpm-bug60179.patch&revision=1320066041
------------------------------------------------------------------------
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=60179
--
Edit this bug report at https://bugs.php.net/bug.php?id=60179&edit=1
