Instead of making them local variables, I'd suggest writing the split data back to $this->options should they need to be reused for anything later (which is the same as what the MySQLi driver does).

Another solution, now the port was checked in the constructor (and saved in $options / $this->options).
But the isset($options['port']) directive has no effect, it was not passed from configuration.php.

I have searched for allowed characters for the hostname (never trust user input) and I've found this: https://tools.ietf.org/html/rfc3986#section-3.2.2 . My solution is not IPv6 compatible.

Ported your solution for postgresql with two exceptions.

1. different way for unix sockets: I have tested it and it works for me. But it is (currently) limited to special characters: (/[a-zA-Z0-9.-_]*)+ . Additionally something like /var/../var/pg is posible (maybe bad).
2. IPv6 doesn't need the [] characters. It works for localhost ::1 but not for a real address (eth0 IPv6) (and not with psql; don't know why).

Please update your PR to latest staging and fix the merge conflicts.

Hello @Joe136

Thank you for your contribution.

The last comment here was on 3rd February. Can you please respond to the feedback given?
If no reply is received within 4 weeks we will close this issue.

Thanks for understanding!

Closed due to lack of response for further information - it can always be reopened if that is provided