2011/12/2 Daniel K. <d...@uw.no>:
> Yasuo Ohgaki wrote:
>>
>> 2011/12/2 Yasuo Ohgaki <yohg...@ohgaki.net>:
>>>
>>> I think Daniel mean there are extra spaces for indent.
>>> I'll fix it.
>
>
> That's exactly it, however the updated patch still has problems.
>
> Search for a + followed by only tabs or spaces. Empty lines should be
> just that, empty.

Does CODING_STANDARDS mention this?

>>> Since Daniel mentioned that he cannot disable strict session,
>
>
> I did no such thing. from where did you get that idea?
>

Because you wrote this.

> This could never work with:
>
> session_id("foo");
> session_start();
>
> could it?

I think you understands it can be controlled by session.use_strict_mode now.


> I am in serious doubt as to whether the additonal restrictions on valid
> characters in session ids are appropriate, and I fear that some poor sod may
> be in for a nasty surpris because of this.
>
> Remember, this is not just about the return value of hash functions, as this
> is used to validate session_ids set with session_id() as well.

With strict session, user cannot set session ID. If user can, it's not
a strict session, but adoptive.

If user would like to use adoptive session, user may set
session.use_strict_mode=0.

Regards,

--
Yasuo Ohgaki
yohg...@ohgaki.net

-- 
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to