Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:58539
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.160.170 as permitted sender)
Received-SPF: pass (google.com: domain of yohgaki@gmail.com designates 10.101.152.32 as permitted sender) client-ip=10.101.152.32;
MIME-Version: 1.0
Sender: yohgaki@gmail.com
In-Reply-To: <4F512530.1020609@sugarcrm.com>
References: <slrnjl05bd.mfm.dsp@experimentalworks.net> <9832D59F-4388-4D50-8D37-A3F74516D51E@stefan-marr.de>
 <4F507B14.8060002@sugarcrm.com> <CAGa2bXZFjuxvFCrWUL_rtXfGsmEMR2bWLytpn+23ZRDK2qra=A@mail.gmail.com>
 <4F512530.1020609@sugarcrm.com>
Date: Sat, 3 Mar 2012 08:48:49 +0900
Message-ID: <CAGa2bXb+Zcri64hyW709g0+1WWnKa03J3B_4RyfO9u-S8-2tGA@mail.gmail.com>
To: Stas Malyshev <smalyshev@sugarcrm.com>
Cc: "internals@lists.php.net" <internals@lists.php.net>
Content-Type: text/plain; charset=ISO-8859-1
Subject: Re: [PHP-DEV] HEADS UP: 5.4 branch is open again
From: yohgaki@ohgaki.net (Yasuo Ohgaki)

Hi Stats

The new strict session patch is not ready, yet.
I'll prepare path for 5.3/5.4/trunk.
I was asked for session id collision detection, so
I'll also add that for create_sid() functions.

>> I've also added 2 new escape functions to pgsql in trunk
>> a while ago, pg_escape_literal()/pg_escape_identifier().
>> Is it okay to merge 5.4 branch also?
>
>
> Should be ok, please send me the commit to review. If it doesn't change any
> structures just adds functions it should be fine.

It just adds new functions. I'll find the commit for review later.

Regards,

--
Yasuo Ohgaki
yohgaki@ohgaki.net



2012/3/3 Stas Malyshev <smalyshev@sugarcrm.com>:
> Hi!
>
>
>> I'll work on the patch for strict session patch so that both
>> PHP 5.4 and 5.3 has the same functionality.
>
>
> OK, do you have the latest patch for that? I remember originally it involved
> some non-BC changes and there were other questions, but I'd like to see the
> final one. In general, if there's no BC issues, I think it may be OK, but
> let's see the final patch.
>
>
>> I've also added 2 new escape functions to pgsql in trunk
>> a while ago, pg_escape_literal()/pg_escape_identifier().
>> Is it okay to merge 5.4 branch also?
>
>
> Should be ok, please send me the commit to review. If it doesn't change any
> structures just adds functions it should be fine.
>
>
> --
> Stanislav Malyshev, Software Architect
> SugarCRM: http://www.sugarcrm.com/
> (408)454-6900 ext. 227