Re: [SM-DEVEL] session_write_close()

[Marc G. K. <ma...@it...>]

Session_write_close, see validate.php in devel.
There were more problems with it and it happens when in php.ini autostart
sessions is set.

In other words, there are only 2 locations in SM where the session is started
and those are redirect and validate.

Marc Groot Koerkamp.


Jason Munro zei:
> Thijs Kinkhorst said:
>> Hello Thomas,
>>
>>> Adding session_write_close(); in front of header("Location:
>>> $redirect_url"); in redirect.php as mentioned
>>> on http://www.php.net/manual/en/function.session-write-close.php did
>>> solve the problem.
>>
>> This seems like a very logical solution to the problem, and if it
>> doesn't  solve anything at least it doesn't break anything, so I
>> committed this to  the source. Let's hope it eliminates some/many of the
>> "You must be logged  in..." problems.
>>
>> Thanks for the research!
>>
>>  >My question: there are about another 30 header("Location: ...");
>>>entries in SM. Wouldn't it be safe to close the session anyway before
>>> redirecting the user?
>>
>> Something to look into indeed.
>> However, redirect.php is one of the parts where the session writing is
>> most  essential.
>>
>
> Thijs,
>   Looks like a good idea, lets see how works in dev and maybe add this fix
> to stable.
>
>
>  \___ Jason Munro
>   \___ AIM:jmunr0
>    \__ ja...@st...
>     \__ http://www.sunflower.com/~jmunro/
>
>
>
>
> -------------------------------------------------------
> This sf.net email is sponsored by:ThinkGeek
> Welcome to geek heaven.
> http://thinkgeek.com/sf
> --
> squirrelmail-devel mailing list
> List Address: squ...@li...
> List Info: https://lists.sourceforge.net/lists/listinfo/squirrelmail-devel
> http://squirrelmail.org/cvs
>