[5.4] allow Redis min server port =0

[alikon]

Pull Request for Issue #46162 .

Summary of Changes

set min value =0 for redis server port

Testing Instructions

see #46162

Actual result BEFORE applying this Pull Request

Invalid field: Redis server port

Expected result AFTER applying this Pull Request

redis connect

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

[richard67]

What happens if you enter zero in the form and a server which is not a path to a socket but just a hostname and save?

Just am curious.

[alikon]

if you need to change the port to zero you probably already know what you are doing and why, there is already a default that works well for the majority of setups, right now you cannot use port zero as for xml, that's all

[richard67]

if you need to change the port to zero you probably already know what you are doing and why, there is already a default that works well for the majority of setups, right now you cannot use port zero as for xml, that's all

@alikon Yes, that's ok, and I do not want to question this PR. I just was curious on what would happen, and maybe someone can test that.

[alikon]

on a standard setup i mean port 6379 (default) if you set zero as port you are not able to connect to the redis server, nothing different than before, this pr just allow to use port zero when you know why and when to do it

[wilsonge]

Is it too much to hide the port field if the connection url is a file ending in .sock? or we overdoing it?

[alikon]

subject for another pull request