[5] add host database port

[alikon]

Draft partial Pull Request for Feature request #43902 .

Summary of Changes

add db host port filed

Testing Instructions

ìnstall and play with different setup db hostname and db port

Actual result BEFORE applying this Pull Request

postgres pdo doesn't work if port different from default (5432)

Expected result AFTER applying this Pull Request

you can set a db port

Link to documentations

Please select:

• Documentation link for docs.joomla.org:

• No documentation changes for docs.joomla.org needed

• Pull Request link for manual.joomla.org:

• No documentation changes for manual.joomla.org needed

[muhme]

Thank you @alikon for this draft PR, I added a language overwrite and did a picture:

There are arguments for (e.g. using 2001:db8::1 and 1066) and against an additional field for the port number (e.g. using [2001:db8::1]:1066). I vote for a new field because two reasons:

1. The documentation. We have to give help on screen and to describe it in detail and this is easier with separate field and not using additional square brackets for IPv6 addresses
2. Security checks with two separate fields for host and port number are easier to implement and maintain as with single field

Why do we need backwards compatibility here? And there is no hurry, from my point of view we can have it 5.2.

Another point: I would make the field optional and blank by default if the default database port is used. This again for simplicity: If we display the default port number, then we have to change it when the database is changed.

[degobbis]

I like the idea behind this PR, but I don't think it makes sense to make the field required.
If the field is empty, the default port should be used automatically.
In addition, it would certainly make sense to check the previous entry in “Host” to see whether a port was already specified there and, if necessary, to move it to the new field and to change the configuration.php.

[alikon]

with joomla-framework/database#310 no more needed

in cypress.config.mjs db_host: 'localhost:5433'