Added PostgreSQL support for non-standard ports (Issue #4152).

[Joe136]

Here is a pull request as described in my previous issue #4152.

I have tested it with and without port definition and both works in my configuration.

[mbabker]

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).

[Joe136]

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.

[Joe136]

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.

[Joe136]

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).

[Hackwar]

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

[roland-d]

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!

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/4165.}

[brianteeman]

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

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/4165.}