Bug #74429 [Asn->Csd]: Remote socket URI with unique persistence identifier broken

Edit report at https://bugs.php.net/bug.php?id=74429&edit=1 ID: 74429 Updated by: ab@php.net Reported by: martijn dot grendelman at isaac dot nl Summary: Remote socket URI with unique persistence identifier broken -Status: Assigned +Status: Closed Type: Bug Package: Streams related Operating System: Any PHP Version: 7.0.18RC1 Assigned To: pollita Block user comment: N Private report: N New Comment: The fix for this bug has been committed. Snapshots of the sources are packaged every three hours; this change will be in the next snapshot. You can grab the snapshot at http://snaps.php.net/. For Windows: http://windows.php.net/snapshots/ Thank you for the report, and for helping us make PHP better. Previous Comments: ------------------------------------------------------------------------ [2017-04-21 13:19:11] requinix@php.net Related To: Bug #74486 ------------------------------------------------------------------------ [2017-04-18 21:40:44] pollita@php.net For what it's worth, https://gist.github.com/sgolemon/51f329655a94523546f081bf9c9afd31 is probably safe enough to use as it would still cover the security case given by bug #74216, it would allow colinmollenhour's hack to keep working, and it's fairly "URI-ish", so it's not an unreasonable looking thing. I'm not personally convinced though. ------------------------------------------------------------------------ [2017-04-18 21:16:29] pollita@php.net No. That "undocumented feature" was never intended to work, and the use of it is an abuse of the API. As to allowing multiple persistent connections to a backend resource, I think we could add official support for it via a new API, and perhaps even matching this use case, but it's going to take some discussion because I don't want to trivially walk into codifying something for no better reason that "It worked on accident at some point." ------------------------------------------------------------------------ [2017-04-18 00:51:55] dominic at varspool dot com This user-contributed note should be amended/removed following an outcome: http://php.net/manual/en/function.stream-socket-client.php#105393 ------------------------------------------------------------------------ [2017-04-18 00:48:13] dominic at varspool dot com Here's another use case (one that I introduced, so I'm feeling guilty about): https://github.com/nrk/predis/pull/139 The problem being that Redis has the concept of multiple databases per server (so, the host, port etc. will match). So, if you want to use a persistent connections, you end up with cross-writes between the databases. (More worked example at that link.) ------------------------------------------------------------------------ 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=74429 -- Edit this bug report at https://bugs.php.net/bug.php?id=74429&edit=1