I think the attached patch maybe slightly better than the one proposed, the
attached patch also accounts for passwords that may contain \ character inside
them.
It doesn't look like any patch was applied and there are conflicts with both Iliaa's and my previous patches. I'm going to merge the patches/conflicts and make a new PR on Github.
Hi,
i don't quite get why we need the PDOTEST_DSN along with PDO_PGSQL_TEST_DSN. Those would differentiate themselves only with the password= param. Maybe just cutoff the password param from PDO_PGSQL_TEST_DSN? Were simpler than having to set PDOTEST_DSN just for this one test. Will need to be fixed somewhen.
Cheers.
[2012-07-04 12:29 UTC] brice at bmaron dot net
[2012-09-17 19:38 UTC] willfitch@php.net
-Status: Open +Status: Verified -Assigned To: +Assigned To: willfitch
[2012-09-17 19:39 UTC] willfitch@php.net
[2012-10-31 01:53 UTC] iliaa@php.net
[2012-10-31 18:47 UTC] willfitch@php.net
[2013-07-22 04:37 UTC] stas@php.net
[2013-11-08 15:09 UTC] willfitch@php.net
[2014-01-01 12:37 UTC] felipe@php.net
-Package: PDO related +Package: PDO PgSQL
[2014-01-08 15:22 UTC] willfitch@php.net
[2014-01-18 22:05 UTC] willfitch@php.net
[2014-01-19 01:40 UTC] ab@php.net
[2014-01-19 01:43 UTC] willfitch@php.net